/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements.  See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You under the Apache License, Version 2.0
 * (the "License"); you may not use this file except in compliance with
 * the License.  You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 *  Unless required by applicable law or agreed to in writing, software
 *  distributed under the License is distributed on an "AS IS" BASIS,
 *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 *  See the License for the specific language governing permissions and
 *  limitations under the License. 
 */

package org.apache.bcel.classfile;

import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.io.IOException;
import java.io.Serializable;

import org.apache.bcel.Constants;

/**
 * Entry of the parameters table.
 * 
 * @see <a href="http://docs.oracle.com/javase/specs/jvms/se8/html/jvms-4.html#jvms-4.7.24">The class File Format : The MethodParameters Attribute</a>
 * @since 6.0
 */
public class MethodParameter implements Serializable, Cloneable {

    private static final long serialVersionUID = 6014494029439440326L;

    /** Index of the CONSTANT_Utf8_info structure in the constant_pool table representing the name of the parameter */
    private int name_index;

    /** The access flags */
    private int access_flags;

    public MethodParameter() {
    }

    /**
     * Construct object from file stream.
     * 
     * @param file Input stream
     * @throws java.io.IOException
     * @throws ClassFormatException
     */
    MethodParameter(DataInputStream file) throws IOException {
        name_index = file.readUnsignedShort();
        access_flags = file.readUnsignedShort();
    }

    public int getNameIndex() {
        return name_index;
    }

    public void setNameIndex(int name_index) {
        this.name_index = name_index;
    }

    /**
     * Returns the name of the parameter.
     */
    public String getParameterName(ConstantPool constant_pool) {
        if (name_index == 0) {
            return null;
        } else {
            return ((ConstantUtf8) constant_pool.getConstant(name_index, Constants.CONSTANT_Utf8)).getBytes();
        }
       }

    public int getAccessFlags() {
        return access_flags;
    }

    public void setAccessFlags(int access_flags) {
        this.access_flags = access_flags;
    }

    public boolean isFinal() {
        return (access_flags & Constants.ACC_FINAL) != 0;
    }

    public boolean isSynthetic() {
        return (access_flags & Constants.ACC_SYNTHETIC) != 0;
    }

    public boolean isMandated() {
        return (access_flags & Constants.ACC_MANDATED) != 0;
    }

    /**
     * Dump object to file stream on binary format.
     *
     * @param file Output file stream
     * @throws IOException
     */
    public final void dump(DataOutputStream file) throws IOException {
        file.writeShort(name_index);
        file.writeShort(access_flags);
    }

    /**
     * @return deep copy of this object
     */
    public MethodParameter copy() {
        try {
            return (MethodParameter) clone();
        } catch (CloneNotSupportedException e) {
        }
        return null;
    }
}

@@ -2277,20 +2277,9 @@
    %7D
- else %7B%0A    
+%0A
     
@@ -2381,26 +2381,16 @@
 ytes();%0A
-        %7D%0A
        %7D

Statement unnecessarily nested within else clause.