You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2019/09/01 22:18:45 UTC

[commons-bcel] branch master updated (97f928d -> 8cabe90)

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git.


    from 97f928d  Cannot break binary compatibility with 6.3.1.
     new 7e8d9a5  Remove redundant modifiers. Minor Javadoc and formatting.
     new 61b8686  Fix spelling in Javadoc and such.
     new 53f2a54  Fix Javadocs.
     new d8c0667  Javadoc.
     new 745d93c  Remove redundant semicolon.
     new 8cabe90  Oops, parent should be version 48, not a snapshot.

The 6 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 src/examples/Mini/ASCII_CharStream.java            | 18 +++++------
 src/examples/Mini/EnvEntry.java                    |  6 ++--
 src/examples/Mini/MiniParserTreeConstants.java     | 24 +++++++--------
 src/examples/Mini/Node.java                        | 16 +++++-----
 src/main/java/org/apache/bcel/classfile/Code.java  | 26 ++++++++--------
 .../org/apache/bcel/classfile/CodeException.java   | 24 +++++++--------
 .../org/apache/bcel/classfile/ConstantClass.java   |  8 ++---
 .../org/apache/bcel/classfile/ConstantDouble.java  |  8 ++---
 .../org/apache/bcel/classfile/ConstantDynamic.java |  4 +--
 .../org/apache/bcel/classfile/ConstantFloat.java   |  8 ++---
 .../org/apache/bcel/classfile/ConstantInteger.java |  8 ++---
 .../bcel/classfile/ConstantInvokeDynamic.java      |  4 +--
 .../org/apache/bcel/classfile/ConstantLong.java    |  8 ++---
 .../bcel/classfile/ConstantMethodHandle.java       |  4 +--
 .../apache/bcel/classfile/ConstantMethodType.java  |  4 +--
 .../org/apache/bcel/classfile/ConstantModule.java  |  8 ++---
 .../apache/bcel/classfile/ConstantNameAndType.java | 16 +++++-----
 .../org/apache/bcel/classfile/ConstantPackage.java |  8 ++---
 .../org/apache/bcel/classfile/ConstantString.java  |  8 ++---
 .../org/apache/bcel/classfile/ConstantUtf8.java    |  8 ++---
 .../org/apache/bcel/classfile/ConstantValue.java   |  8 ++---
 .../java/org/apache/bcel/classfile/Deprecated.java |  8 ++---
 .../org/apache/bcel/classfile/ExceptionTable.java  | 12 ++++----
 src/main/java/org/apache/bcel/classfile/Field.java |  6 ++--
 .../java/org/apache/bcel/classfile/InnerClass.java | 22 ++++++-------
 .../org/apache/bcel/classfile/InnerClasses.java    |  8 ++---
 .../java/org/apache/bcel/classfile/LineNumber.java | 12 ++++----
 .../org/apache/bcel/classfile/LineNumberTable.java | 10 +++---
 .../org/apache/bcel/classfile/LocalVariable.java   | 36 +++++++++++-----------
 .../java/org/apache/bcel/classfile/Method.java     | 12 ++++----
 .../java/org/apache/bcel/classfile/Module.java     | 12 ++++----
 .../org/apache/bcel/classfile/ModuleExports.java   |  6 ++--
 .../org/apache/bcel/classfile/ModuleMainClass.java | 16 +++++-----
 .../org/apache/bcel/classfile/ModuleOpens.java     |  6 ++--
 .../org/apache/bcel/classfile/ModulePackages.java  | 12 ++++----
 .../org/apache/bcel/classfile/ModuleProvides.java  |  6 ++--
 .../org/apache/bcel/classfile/ModuleRequires.java  |  6 ++--
 .../java/org/apache/bcel/classfile/NestHost.java   | 20 ++++++------
 .../org/apache/bcel/classfile/NestMembers.java     | 12 ++++----
 .../java/org/apache/bcel/classfile/PMGClass.java   | 16 +++++-----
 .../java/org/apache/bcel/classfile/Signature.java  | 14 ++++-----
 .../java/org/apache/bcel/classfile/SourceFile.java | 10 +++---
 .../java/org/apache/bcel/classfile/StackMap.java   | 10 +++---
 .../org/apache/bcel/classfile/StackMapEntry.java   |  8 ++---
 .../org/apache/bcel/classfile/StackMapType.java    | 10 +++---
 .../java/org/apache/bcel/classfile/Synthetic.java  |  8 ++---
 .../java/org/apache/bcel/classfile/Unknown.java    | 10 +++---
 src/main/java/org/apache/bcel/generic/PUSH.java    |  4 +--
 src/main/java/org/apache/bcel/generic/SWITCH.java  |  4 +--
 .../java/org/apache/bcel/util/AttributeHTML.java   |  6 ++--
 .../java/org/apache/bcel/util/ByteSequence.java    |  8 ++---
 src/test/java/org/apache/bcel/PerformanceTest.java |  2 +-
 52 files changed, 279 insertions(+), 279 deletions(-)


[commons-bcel] 04/06: Javadoc.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit d8c066732565ff079c7ba8c6bea8774bb07a5bfc
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:15:42 2019 -0400

    Javadoc.
---
 src/main/java/org/apache/bcel/classfile/NestHost.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/NestHost.java b/src/main/java/org/apache/bcel/classfile/NestHost.java
index b261148..b343088 100644
--- a/src/main/java/org/apache/bcel/classfile/NestHost.java
+++ b/src/main/java/org/apache/bcel/classfile/NestHost.java
@@ -36,7 +36,7 @@ public final class NestHost extends Attribute {
 
 
     /**
-     * Initialize from another object. Note that both objects use the same
+     * Initializes from another object. Note that both objects use the same
      * references (shallow copy). Use copy() for a physical copy.
      */
     public NestHost(final NestHost c) {
@@ -58,7 +58,7 @@ public final class NestHost extends Attribute {
 
 
     /**
-     * Construct object from input stream.
+     * Constructs object from input stream.
      * @param name_index Index in constant pool
      * @param length Content length in bytes
      * @param input Input stream
@@ -85,7 +85,7 @@ public final class NestHost extends Attribute {
 
 
     /**
-     * Dump NestHost attribute to file stream in binary format.
+     * Dumps NestHost attribute to file stream in binary format.
      *
      * @param file Output file stream
      * @throws IOException if an I/O error occurs.


[commons-bcel] 01/06: Remove redundant modifiers. Minor Javadoc and formatting.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit 7e8d9a5529854590bcf293f675a37ac309f699c0
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:11:49 2019 -0400

    Remove redundant modifiers. Minor Javadoc and formatting.
---
 pom.xml                                            |  2 +-
 src/examples/Mini/ASCII_CharStream.java            | 18 +++++------
 src/examples/Mini/EnvEntry.java                    |  6 ++--
 src/examples/Mini/MiniParserTreeConstants.java     | 24 +++++++--------
 src/examples/Mini/Node.java                        | 16 +++++-----
 src/main/java/org/apache/bcel/classfile/Code.java  | 26 ++++++++--------
 .../org/apache/bcel/classfile/CodeException.java   | 24 +++++++--------
 .../org/apache/bcel/classfile/ConstantClass.java   |  8 ++---
 .../org/apache/bcel/classfile/ConstantDouble.java  |  8 ++---
 .../org/apache/bcel/classfile/ConstantDynamic.java |  4 +--
 .../org/apache/bcel/classfile/ConstantFloat.java   |  8 ++---
 .../org/apache/bcel/classfile/ConstantInteger.java |  8 ++---
 .../bcel/classfile/ConstantInvokeDynamic.java      |  4 +--
 .../org/apache/bcel/classfile/ConstantLong.java    |  8 ++---
 .../bcel/classfile/ConstantMethodHandle.java       |  4 +--
 .../apache/bcel/classfile/ConstantMethodType.java  |  4 +--
 .../org/apache/bcel/classfile/ConstantModule.java  |  8 ++---
 .../apache/bcel/classfile/ConstantNameAndType.java | 16 +++++-----
 .../org/apache/bcel/classfile/ConstantPackage.java |  8 ++---
 .../org/apache/bcel/classfile/ConstantString.java  |  8 ++---
 .../org/apache/bcel/classfile/ConstantUtf8.java    |  8 ++---
 .../org/apache/bcel/classfile/ConstantValue.java   |  8 ++---
 .../java/org/apache/bcel/classfile/Deprecated.java |  8 ++---
 .../org/apache/bcel/classfile/ExceptionTable.java  | 12 ++++----
 src/main/java/org/apache/bcel/classfile/Field.java |  6 ++--
 .../java/org/apache/bcel/classfile/InnerClass.java | 22 ++++++-------
 .../org/apache/bcel/classfile/InnerClasses.java    |  8 ++---
 .../java/org/apache/bcel/classfile/LineNumber.java | 12 ++++----
 .../org/apache/bcel/classfile/LineNumberTable.java | 10 +++---
 .../org/apache/bcel/classfile/LocalVariable.java   | 36 +++++++++++-----------
 .../java/org/apache/bcel/classfile/Method.java     | 12 ++++----
 .../java/org/apache/bcel/classfile/Module.java     | 12 ++++----
 .../org/apache/bcel/classfile/ModuleExports.java   |  6 ++--
 .../org/apache/bcel/classfile/ModuleMainClass.java |  8 ++---
 .../org/apache/bcel/classfile/ModuleOpens.java     |  6 ++--
 .../org/apache/bcel/classfile/ModulePackages.java  | 12 ++++----
 .../org/apache/bcel/classfile/ModuleProvides.java  |  6 ++--
 .../org/apache/bcel/classfile/ModuleRequires.java  |  6 ++--
 .../java/org/apache/bcel/classfile/NestHost.java   |  8 ++---
 .../org/apache/bcel/classfile/NestMembers.java     | 12 ++++----
 .../java/org/apache/bcel/classfile/PMGClass.java   | 16 +++++-----
 .../java/org/apache/bcel/classfile/Signature.java  | 14 ++++-----
 .../java/org/apache/bcel/classfile/SourceFile.java | 10 +++---
 .../java/org/apache/bcel/classfile/StackMap.java   | 10 +++---
 .../org/apache/bcel/classfile/StackMapEntry.java   |  8 ++---
 .../org/apache/bcel/classfile/StackMapType.java    | 10 +++---
 .../java/org/apache/bcel/classfile/Synthetic.java  |  8 ++---
 .../java/org/apache/bcel/classfile/Unknown.java    | 10 +++---
 src/main/java/org/apache/bcel/generic/PUSH.java    |  4 +--
 src/main/java/org/apache/bcel/generic/SWITCH.java  |  4 +--
 .../java/org/apache/bcel/util/AttributeHTML.java   |  6 ++--
 .../java/org/apache/bcel/util/ByteSequence.java    |  8 ++---
 52 files changed, 269 insertions(+), 269 deletions(-)

diff --git a/pom.xml b/pom.xml
index 76f468f..21446e1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -27,7 +27,7 @@
   <parent>
     <groupId>org.apache.commons</groupId>
     <artifactId>commons-parent</artifactId>
-    <version>48</version>
+    <version>49-SNAPSHOT</version>
   </parent>
 
   <groupId>org.apache.bcel</groupId>
diff --git a/src/examples/Mini/ASCII_CharStream.java b/src/examples/Mini/ASCII_CharStream.java
index f2398b5..943c51f 100644
--- a/src/examples/Mini/ASCII_CharStream.java
+++ b/src/examples/Mini/ASCII_CharStream.java
@@ -143,7 +143,7 @@ public final class ASCII_CharStream
      }
   }
 
-  static public final char BeginToken() throws java.io.IOException
+  static public char BeginToken() throws java.io.IOException
   {
      tokenBegin = -1;
      final char c = readChar();
@@ -192,7 +192,7 @@ public final class ASCII_CharStream
      bufcolumn[bufpos] = column;
   }
 
-  static public final char readChar() throws java.io.IOException
+  static public char readChar() throws java.io.IOException
   {
      if (inBuf > 0)
      {
@@ -210,23 +210,23 @@ public final class ASCII_CharStream
      return (c);
   }
 
-  static public final int getEndColumn() {
+  static public int getEndColumn() {
      return bufcolumn[bufpos];
   }
 
-  static public final int getEndLine() {
+  static public int getEndLine() {
      return bufline[bufpos];
   }
 
-  static public final int getBeginColumn() {
+  static public int getBeginColumn() {
      return bufcolumn[tokenBegin];
   }
 
-  static public final int getBeginLine() {
+  static public int getBeginLine() {
      return bufline[tokenBegin];
   }
 
-  static public final void backup(final int amount) {
+  static public void backup(final int amount) {
 
     inBuf += amount;
     if ((bufpos -= amount) < 0) {
@@ -303,7 +303,7 @@ public final class ASCII_CharStream
   {
      ReInit(dstream, startline, startcolumn, 4096);
   }
-  static public final String GetImage()
+  static public String GetImage()
   {
      if (bufpos >= tokenBegin) {
         return new String(buffer, tokenBegin, bufpos - tokenBegin + 1);
@@ -313,7 +313,7 @@ public final class ASCII_CharStream
     }
   }
 
-  static public final char[] GetSuffix(final int len)
+  static public char[] GetSuffix(final int len)
   {
      final char[] ret = new char[len];
 
diff --git a/src/examples/Mini/EnvEntry.java b/src/examples/Mini/EnvEntry.java
index 32d5bc4..16637b7 100644
--- a/src/examples/Mini/EnvEntry.java
+++ b/src/examples/Mini/EnvEntry.java
@@ -22,7 +22,7 @@ package Mini;
  *
  */
 public interface EnvEntry {
-  public String getHashKey();
-  public int    getLine();
-  public int    getColumn();
+  String getHashKey();
+  int    getLine();
+  int    getColumn();
 }
diff --git a/src/examples/Mini/MiniParserTreeConstants.java b/src/examples/Mini/MiniParserTreeConstants.java
index fe74014..535fb62 100644
--- a/src/examples/Mini/MiniParserTreeConstants.java
+++ b/src/examples/Mini/MiniParserTreeConstants.java
@@ -21,20 +21,20 @@ package Mini;
 
 public interface MiniParserTreeConstants
 {
-  public int JJTPROGRAM = 0;
-  public int JJTFUNDECL = 1;
-  public int JJTEXPR = 2;
-  public int JJTIFEXPR = 3;
-  public int JJTLETEXPR = 4;
-  public int JJTFUNAPPL = 5;
-  public int JJTTERM = 6;
-  public int JJTFACTOR = 7;
-  public int JJTVOID = 8;
-  public int JJTINTEGER = 9;
-  public int JJTIDENT = 10;
+  int JJTPROGRAM = 0;
+  int JJTFUNDECL = 1;
+  int JJTEXPR = 2;
+  int JJTIFEXPR = 3;
+  int JJTLETEXPR = 4;
+  int JJTFUNAPPL = 5;
+  int JJTTERM = 6;
+  int JJTFACTOR = 7;
+  int JJTVOID = 8;
+  int JJTINTEGER = 9;
+  int JJTIDENT = 10;
 
 
-  public String[] jjtNodeName = {
+  String[] jjtNodeName = {
     "Program",
     "FunDecl",
     "Expr",
diff --git a/src/examples/Mini/Node.java b/src/examples/Mini/Node.java
index efbfe88..18cc9cb 100644
--- a/src/examples/Mini/Node.java
+++ b/src/examples/Mini/Node.java
@@ -20,32 +20,32 @@
 
 package Mini;
 
-/* All AST nodes must implement this interface.  It provides basic
+/** All AST nodes must implement this interface.  It provides basic
    machinery for constructing the parent and child relationships
    between nodes. */
-
 public interface Node {
 
   /** This method is called after the node has been made the current
     node.  It indicates that child nodes can now be added to it. */
-  public void jjtOpen();
+  void jjtOpen();
 
   /** This method is called after all the child nodes have been
     added. */
-  public void jjtClose();
+  void jjtClose();
 
   /** This pair of methods are used to inform the node of its
     parent. */
-  public void jjtSetParent(Node n);
-  public Node jjtGetParent();
+  void jjtSetParent(Node n);
+  
+  Node jjtGetParent();
 
   /** This method tells the node to add its argument to the node's
     list of children.  */
-  public void jjtAddChild(Node n, int i);
+  void jjtAddChild(Node n, int i);
 
   /** This method returns a child node.  The children are numbered
      from zero, left to right. */
-  public Node jjtGetChild(int i);
+  Node jjtGetChild(int i);
 
   /** Return the number of children the node has. */
   int jjtGetNumChildren();
diff --git a/src/main/java/org/apache/bcel/classfile/Code.java b/src/main/java/org/apache/bcel/classfile/Code.java
index 50962d0..9525574 100644
--- a/src/main/java/org/apache/bcel/classfile/Code.java
+++ b/src/main/java/org/apache/bcel/classfile/Code.java
@@ -139,7 +139,7 @@ public final class Code extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(max_stack);
         file.writeShort(max_locals);
@@ -160,7 +160,7 @@ public final class Code extends Attribute {
      * @return Collection of code attributes.
      * @see Attribute
      */
-    public final Attribute[] getAttributes() {
+    public Attribute[] getAttributes() {
         return attributes;
     }
 
@@ -194,7 +194,7 @@ public final class Code extends Attribute {
     /**
      * @return Actual byte code of the method.
      */
-    public final byte[] getCode() {
+    public byte[] getCode() {
         return code;
     }
 
@@ -203,7 +203,7 @@ public final class Code extends Attribute {
      * @return Table of handled exceptions.
      * @see CodeException
      */
-    public final CodeException[] getExceptionTable() {
+    public CodeException[] getExceptionTable() {
         return exception_table;
     }
 
@@ -211,7 +211,7 @@ public final class Code extends Attribute {
     /**
      * @return Number of local variables.
      */
-    public final int getMaxLocals() {
+    public int getMaxLocals() {
         return max_locals;
     }
 
@@ -219,7 +219,7 @@ public final class Code extends Attribute {
     /**
      * @return Maximum size of stack used by this method.
      */
-    public final int getMaxStack() {
+    public int getMaxStack() {
         return max_stack;
     }
 
@@ -255,7 +255,7 @@ public final class Code extends Attribute {
     /**
      * @param attributes the attributes to set for this Code
      */
-    public final void setAttributes( final Attribute[] attributes ) {
+    public void setAttributes( final Attribute[] attributes ) {
         this.attributes = attributes != null ? attributes : new Attribute[0];
         super.setLength(calculateLength()); // Adjust length
     }
@@ -264,7 +264,7 @@ public final class Code extends Attribute {
     /**
      * @param code byte code
      */
-    public final void setCode( final byte[] code ) {
+    public void setCode( final byte[] code ) {
         this.code = code != null ? code : new byte[0];
         super.setLength(calculateLength()); // Adjust length
     }
@@ -273,7 +273,7 @@ public final class Code extends Attribute {
     /**
      * @param exception_table exception table
      */
-    public final void setExceptionTable( final CodeException[] exception_table ) {
+    public void setExceptionTable( final CodeException[] exception_table ) {
         this.exception_table = exception_table != null ? exception_table : new CodeException[0];
         super.setLength(calculateLength()); // Adjust length
     }
@@ -282,7 +282,7 @@ public final class Code extends Attribute {
     /**
      * @param max_locals maximum number of local variables
      */
-    public final void setMaxLocals( final int max_locals ) {
+    public void setMaxLocals( final int max_locals ) {
         this.max_locals = max_locals;
     }
 
@@ -290,7 +290,7 @@ public final class Code extends Attribute {
     /**
      * @param max_stack maximum stack size
      */
-    public final void setMaxStack( final int max_stack ) {
+    public void setMaxStack( final int max_stack ) {
         this.max_stack = max_stack;
     }
 
@@ -298,7 +298,7 @@ public final class Code extends Attribute {
     /**
      * @return String representation of code chunk.
      */
-    public final String toString( final boolean verbose ) {
+    public String toString( final boolean verbose ) {
         final StringBuilder buf = new StringBuilder(100); // CHECKSTYLE IGNORE MagicNumber
         buf.append("Code(max_stack = ").append(max_stack).append(", max_locals = ").append(
                 max_locals).append(", code_length = ").append(code.length).append(")\n").append(
@@ -323,7 +323,7 @@ public final class Code extends Attribute {
      * @return String representation of code chunk.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return toString(true);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/CodeException.java b/src/main/java/org/apache/bcel/classfile/CodeException.java
index 39e0e18..c79cc9b 100644
--- a/src/main/java/org/apache/bcel/classfile/CodeException.java
+++ b/src/main/java/org/apache/bcel/classfile/CodeException.java
@@ -100,7 +100,7 @@ public final class CodeException implements Cloneable, Node, Constants {
      * @param file Output file stream
      * @throws IOException
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(start_pc);
         file.writeShort(end_pc);
         file.writeShort(handler_pc);
@@ -112,7 +112,7 @@ public final class CodeException implements Cloneable, Node, Constants {
      * @return 0, if the handler catches any exception, otherwise it points to
      * the exception class which is to be caught.
      */
-    public final int getCatchType() {
+    public int getCatchType() {
         return catch_type;
     }
 
@@ -120,7 +120,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @return Exclusive end index of the region where the handler is active.
      */
-    public final int getEndPC() {
+    public int getEndPC() {
         return end_pc;
     }
 
@@ -128,7 +128,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @return Starting address of exception handler, relative to the code.
      */
-    public final int getHandlerPC() {
+    public int getHandlerPC() {
         return handler_pc;
     }
 
@@ -136,7 +136,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @return Inclusive start index of the region where the handler is active.
      */
-    public final int getStartPC() {
+    public int getStartPC() {
         return start_pc;
     }
 
@@ -144,7 +144,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @param catch_type the type of exception that is caught
      */
-    public final void setCatchType( final int catch_type ) {
+    public void setCatchType( final int catch_type ) {
         this.catch_type = catch_type;
     }
 
@@ -152,7 +152,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @param end_pc end of handled block
      */
-    public final void setEndPC( final int end_pc ) {
+    public void setEndPC( final int end_pc ) {
         this.end_pc = end_pc;
     }
 
@@ -160,7 +160,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @param handler_pc where the actual code is
      */
-    public final void setHandlerPC( final int handler_pc ) { // TODO unused
+    public void setHandlerPC( final int handler_pc ) { // TODO unused
         this.handler_pc = handler_pc;
     }
 
@@ -168,7 +168,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @param start_pc start of handled block
      */
-    public final void setStartPC( final int start_pc ) { // TODO unused
+    public void setStartPC( final int start_pc ) { // TODO unused
         this.start_pc = start_pc;
     }
 
@@ -177,7 +177,7 @@ public final class CodeException implements Cloneable, Node, Constants {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "CodeException(start_pc = " + start_pc + ", end_pc = " + end_pc + ", handler_pc = "
                 + handler_pc + ", catch_type = " + catch_type + ")";
     }
@@ -186,7 +186,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     /**
      * @return String representation.
      */
-    public final String toString( final ConstantPool cp, final boolean verbose ) {
+    public String toString( final ConstantPool cp, final boolean verbose ) {
         String str;
         if (catch_type == 0) {
             str = "<Any exception>(0)";
@@ -198,7 +198,7 @@ public final class CodeException implements Cloneable, Node, Constants {
     }
 
 
-    public final String toString( final ConstantPool cp ) {
+    public String toString( final ConstantPool cp ) {
         return toString(cp, true);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantClass.java b/src/main/java/org/apache/bcel/classfile/ConstantClass.java
index 25ebfab..0820c18 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantClass.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantClass.java
@@ -83,7 +83,7 @@ public final class ConstantClass extends Constant implements ConstantObject {
      * @throws IOException if an I/O error occurs writing to the DataOutputStream.
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(name_index);
     }
@@ -92,7 +92,7 @@ public final class ConstantClass extends Constant implements ConstantObject {
     /**
      * @return Name index in constant pool of class name.
      */
-    public final int getNameIndex() {
+    public int getNameIndex() {
         return name_index;
     }
 
@@ -100,7 +100,7 @@ public final class ConstantClass extends Constant implements ConstantObject {
     /**
      * @param name_index the name index in the constant pool of this Constant Class
      */
-    public final void setNameIndex( final int name_index ) {
+    public void setNameIndex( final int name_index ) {
         this.name_index = name_index;
     }
 
@@ -125,7 +125,7 @@ public final class ConstantClass extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(name_index = " + name_index + ")";
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantDouble.java b/src/main/java/org/apache/bcel/classfile/ConstantDouble.java
index 73736d4..9ef7ce6 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantDouble.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantDouble.java
@@ -82,7 +82,7 @@ public final class ConstantDouble extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeDouble(bytes);
     }
@@ -91,7 +91,7 @@ public final class ConstantDouble extends Constant implements ConstantObject {
     /**
      * @return data, i.e., 8 bytes.
      */
-    public final double getBytes() {
+    public double getBytes() {
         return bytes;
     }
 
@@ -99,7 +99,7 @@ public final class ConstantDouble extends Constant implements ConstantObject {
     /**
      * @param bytes the raw bytes that represent the double value
      */
-    public final void setBytes( final double bytes ) {
+    public void setBytes( final double bytes ) {
         this.bytes = bytes;
     }
 
@@ -108,7 +108,7 @@ public final class ConstantDouble extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(bytes = " + bytes + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantDynamic.java b/src/main/java/org/apache/bcel/classfile/ConstantDynamic.java
index df3add1..b76db6d 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantDynamic.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantDynamic.java
@@ -76,7 +76,7 @@ public final class ConstantDynamic extends ConstantCP {
      * for use by ConstantInvokeDynamic.
      * @since 6.0
      */
-    public final int getBootstrapMethodAttrIndex() {
+    public int getBootstrapMethodAttrIndex() {
         return super.getClassIndex();  // AKA bootstrap_method_attr_index
     }
 
@@ -84,7 +84,7 @@ public final class ConstantDynamic extends ConstantCP {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString().replace("class_index", "bootstrap_method_attr_index");
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantFloat.java b/src/main/java/org/apache/bcel/classfile/ConstantFloat.java
index eabd94d..569021d 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantFloat.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantFloat.java
@@ -83,7 +83,7 @@ public final class ConstantFloat extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeFloat(bytes);
     }
@@ -92,7 +92,7 @@ public final class ConstantFloat extends Constant implements ConstantObject {
     /**
      * @return data, i.e., 4 bytes.
      */
-    public final float getBytes() {
+    public float getBytes() {
         return bytes;
     }
 
@@ -100,7 +100,7 @@ public final class ConstantFloat extends Constant implements ConstantObject {
     /**
      * @param bytes the raw bytes that represent this float
      */
-    public final void setBytes( final float bytes ) {
+    public void setBytes( final float bytes ) {
         this.bytes = bytes;
     }
 
@@ -109,7 +109,7 @@ public final class ConstantFloat extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(bytes = " + bytes + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantInteger.java b/src/main/java/org/apache/bcel/classfile/ConstantInteger.java
index bc20963..68b524d 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantInteger.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantInteger.java
@@ -82,7 +82,7 @@ public final class ConstantInteger extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeInt(bytes);
     }
@@ -91,7 +91,7 @@ public final class ConstantInteger extends Constant implements ConstantObject {
     /**
      * @return data, i.e., 4 bytes.
      */
-    public final int getBytes() {
+    public int getBytes() {
         return bytes;
     }
 
@@ -99,7 +99,7 @@ public final class ConstantInteger extends Constant implements ConstantObject {
     /**
      * @param bytes the raw bytes that represent this integer
      */
-    public final void setBytes( final int bytes ) {
+    public void setBytes( final int bytes ) {
         this.bytes = bytes;
     }
 
@@ -108,7 +108,7 @@ public final class ConstantInteger extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(bytes = " + bytes + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantInvokeDynamic.java b/src/main/java/org/apache/bcel/classfile/ConstantInvokeDynamic.java
index bc3e94c..91c3b98 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantInvokeDynamic.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantInvokeDynamic.java
@@ -76,7 +76,7 @@ public final class ConstantInvokeDynamic extends ConstantCP {
      * for use by ConstantInvokeDynamic.
      * @since 6.0
      */
-    public final int getBootstrapMethodAttrIndex() {
+    public int getBootstrapMethodAttrIndex() {
         return super.getClassIndex();  // AKA bootstrap_method_attr_index
     }
 
@@ -84,7 +84,7 @@ public final class ConstantInvokeDynamic extends ConstantCP {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString().replace("class_index", "bootstrap_method_attr_index");
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantLong.java b/src/main/java/org/apache/bcel/classfile/ConstantLong.java
index 43a59b2..821c339 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantLong.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantLong.java
@@ -82,7 +82,7 @@ public final class ConstantLong extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeLong(bytes);
     }
@@ -91,7 +91,7 @@ public final class ConstantLong extends Constant implements ConstantObject {
     /**
      * @return data, i.e., 8 bytes.
      */
-    public final long getBytes() {
+    public long getBytes() {
         return bytes;
     }
 
@@ -99,7 +99,7 @@ public final class ConstantLong extends Constant implements ConstantObject {
     /**
      * @param bytes the raw bytes that represent this long
      */
-    public final void setBytes( final long bytes ) {
+    public void setBytes( final long bytes ) {
         this.bytes = bytes;
     }
 
@@ -108,7 +108,7 @@ public final class ConstantLong extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(bytes = " + bytes + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantMethodHandle.java b/src/main/java/org/apache/bcel/classfile/ConstantMethodHandle.java
index 6de1b34..f262958 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantMethodHandle.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantMethodHandle.java
@@ -82,7 +82,7 @@ public final class ConstantMethodHandle extends Constant {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeByte(reference_kind);
         file.writeShort(reference_index);
@@ -113,7 +113,7 @@ public final class ConstantMethodHandle extends Constant {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(reference_kind = " + reference_kind +
                 ", reference_index = " + reference_index + ")";
     }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantMethodType.java b/src/main/java/org/apache/bcel/classfile/ConstantMethodType.java
index 26f832a..f2c1e9d 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantMethodType.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantMethodType.java
@@ -80,7 +80,7 @@ public final class ConstantMethodType extends Constant {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(descriptor_index);
     }
@@ -100,7 +100,7 @@ public final class ConstantMethodType extends Constant {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(descriptor_index = " + descriptor_index + ")";
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantModule.java b/src/main/java/org/apache/bcel/classfile/ConstantModule.java
index 2d746ae..f1621a5 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantModule.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantModule.java
@@ -86,7 +86,7 @@ public final class ConstantModule extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(name_index);
     }
@@ -95,7 +95,7 @@ public final class ConstantModule extends Constant implements ConstantObject {
     /**
      * @return Name index in constant pool of module name.
      */
-    public final int getNameIndex() {
+    public int getNameIndex() {
         return name_index;
     }
 
@@ -103,7 +103,7 @@ public final class ConstantModule extends Constant implements ConstantObject {
     /**
      * @param name_index the name index in the constant pool of this Constant Module
      */
-    public final void setNameIndex( final int name_index ) {
+    public void setNameIndex( final int name_index ) {
         this.name_index = name_index;
     }
 
@@ -128,7 +128,7 @@ public final class ConstantModule extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(name_index = " + name_index + ")";
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantNameAndType.java b/src/main/java/org/apache/bcel/classfile/ConstantNameAndType.java
index eb2ec2b..90645bf 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantNameAndType.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantNameAndType.java
@@ -86,7 +86,7 @@ public final class ConstantNameAndType extends Constant {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(name_index);
         file.writeShort(signature_index);
@@ -96,14 +96,14 @@ public final class ConstantNameAndType extends Constant {
     /**
      * @return Name index in constant pool of field/method name.
      */
-    public final int getNameIndex() {
+    public int getNameIndex() {
         return name_index;
     }
 
 
     /** @return name
      */
-    public final String getName( final ConstantPool cp ) {
+    public String getName( final ConstantPool cp ) {
         return cp.constantToString(getNameIndex(), Const.CONSTANT_Utf8);
     }
 
@@ -111,14 +111,14 @@ public final class ConstantNameAndType extends Constant {
     /**
      * @return Index in constant pool of field/method signature.
      */
-    public final int getSignatureIndex() {
+    public int getSignatureIndex() {
         return signature_index;
     }
 
 
     /** @return signature
      */
-    public final String getSignature( final ConstantPool cp ) {
+    public String getSignature( final ConstantPool cp ) {
         return cp.constantToString(getSignatureIndex(), Const.CONSTANT_Utf8);
     }
 
@@ -126,7 +126,7 @@ public final class ConstantNameAndType extends Constant {
     /**
      * @param name_index the name index of this constant
      */
-    public final void setNameIndex( final int name_index ) {
+    public void setNameIndex( final int name_index ) {
         this.name_index = name_index;
     }
 
@@ -134,7 +134,7 @@ public final class ConstantNameAndType extends Constant {
     /**
      * @param signature_index the signature index in the constant pool of this type
      */
-    public final void setSignatureIndex( final int signature_index ) {
+    public void setSignatureIndex( final int signature_index ) {
         this.signature_index = signature_index;
     }
 
@@ -143,7 +143,7 @@ public final class ConstantNameAndType extends Constant {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(name_index = " + name_index + ", signature_index = "
                 + signature_index + ")";
     }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantPackage.java b/src/main/java/org/apache/bcel/classfile/ConstantPackage.java
index ee01317..b3fd233 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantPackage.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantPackage.java
@@ -86,7 +86,7 @@ public final class ConstantPackage extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(name_index);
     }
@@ -95,7 +95,7 @@ public final class ConstantPackage extends Constant implements ConstantObject {
     /**
      * @return Name index in constant pool of package name.
      */
-    public final int getNameIndex() {
+    public int getNameIndex() {
         return name_index;
     }
 
@@ -103,7 +103,7 @@ public final class ConstantPackage extends Constant implements ConstantObject {
     /**
      * @param name_index the name index in the constant pool of this Constant Package
      */
-    public final void setNameIndex( final int name_index ) {
+    public void setNameIndex( final int name_index ) {
         this.name_index = name_index;
     }
 
@@ -128,7 +128,7 @@ public final class ConstantPackage extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(name_index = " + name_index + ")";
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantString.java b/src/main/java/org/apache/bcel/classfile/ConstantString.java
index 93aa969..a4d2e0c 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantString.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantString.java
@@ -82,7 +82,7 @@ public final class ConstantString extends Constant implements ConstantObject {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(super.getTag());
         file.writeShort(string_index);
     }
@@ -91,7 +91,7 @@ public final class ConstantString extends Constant implements ConstantObject {
     /**
      * @return Index in constant pool of the string (ConstantUtf8).
      */
-    public final int getStringIndex() {
+    public int getStringIndex() {
         return string_index;
     }
 
@@ -99,7 +99,7 @@ public final class ConstantString extends Constant implements ConstantObject {
     /**
      * @param string_index the index into the constant of the string value
      */
-    public final void setStringIndex( final int string_index ) {
+    public void setStringIndex( final int string_index ) {
         this.string_index = string_index;
     }
 
@@ -108,7 +108,7 @@ public final class ConstantString extends Constant implements ConstantObject {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(string_index = " + string_index + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java b/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
index b941981..6ac696e 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
@@ -236,7 +236,7 @@ public final class ConstantUtf8 extends Constant {
      * @throws IOException
      */
     @Override
-    public final void dump(final DataOutputStream file) throws IOException {
+    public void dump(final DataOutputStream file) throws IOException {
         file.writeByte(super.getTag());
         file.writeUTF(value);
     }
@@ -244,7 +244,7 @@ public final class ConstantUtf8 extends Constant {
     /**
      * @return Data converted to string.
      */
-    public final String getBytes() {
+    public String getBytes() {
         return value;
     }
 
@@ -253,7 +253,7 @@ public final class ConstantUtf8 extends Constant {
      * @deprecated (since 6.0)
      */
     @java.lang.Deprecated
-    public final void setBytes(final String bytes) {
+    public void setBytes(final String bytes) {
         throw new UnsupportedOperationException();
     }
 
@@ -261,7 +261,7 @@ public final class ConstantUtf8 extends Constant {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return super.toString() + "(\"" + Utility.replace(value, "\n", "\\n") + "\")";
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/ConstantValue.java b/src/main/java/org/apache/bcel/classfile/ConstantValue.java
index 2bccb72..50ff7c9 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantValue.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantValue.java
@@ -91,7 +91,7 @@ public final class ConstantValue extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(constantvalue_index);
     }
@@ -100,7 +100,7 @@ public final class ConstantValue extends Attribute {
     /**
      * @return Index in constant pool of constant value.
      */
-    public final int getConstantValueIndex() {
+    public int getConstantValueIndex() {
         return constantvalue_index;
     }
 
@@ -108,7 +108,7 @@ public final class ConstantValue extends Attribute {
     /**
      * @param constantvalue_index the index info the constant pool of this constant value
      */
-    public final void setConstantValueIndex( final int constantvalue_index ) {
+    public void setConstantValueIndex( final int constantvalue_index ) {
         this.constantvalue_index = constantvalue_index;
     }
 
@@ -117,7 +117,7 @@ public final class ConstantValue extends Attribute {
      * @return String representation of constant value.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         Constant c = super.getConstantPool().getConstant(constantvalue_index);
         String buf;
         int i;
diff --git a/src/main/java/org/apache/bcel/classfile/Deprecated.java b/src/main/java/org/apache/bcel/classfile/Deprecated.java
index f85b441..0b67e86 100644
--- a/src/main/java/org/apache/bcel/classfile/Deprecated.java
+++ b/src/main/java/org/apache/bcel/classfile/Deprecated.java
@@ -96,7 +96,7 @@ public final class Deprecated extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         if (super.getLength() > 0) {
             file.write(bytes, 0, super.getLength());
@@ -107,7 +107,7 @@ public final class Deprecated extends Attribute {
     /**
      * @return data bytes.
      */
-    public final byte[] getBytes() {
+    public byte[] getBytes() {
         return bytes;
     }
 
@@ -115,7 +115,7 @@ public final class Deprecated extends Attribute {
     /**
      * @param bytes the raw bytes that represents this byte array
      */
-    public final void setBytes( final byte[] bytes ) {
+    public void setBytes( final byte[] bytes ) {
         this.bytes = bytes;
     }
 
@@ -124,7 +124,7 @@ public final class Deprecated extends Attribute {
      * @return attribute name
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return Const.getAttributeName(Const.ATTR_DEPRECATED);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/ExceptionTable.java b/src/main/java/org/apache/bcel/classfile/ExceptionTable.java
index 917b219..2d50c0c 100644
--- a/src/main/java/org/apache/bcel/classfile/ExceptionTable.java
+++ b/src/main/java/org/apache/bcel/classfile/ExceptionTable.java
@@ -98,7 +98,7 @@ public final class ExceptionTable extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(exception_index_table.length);
         for (final int index : exception_index_table) {
@@ -110,7 +110,7 @@ public final class ExceptionTable extends Attribute {
     /**
      * @return Array of indices into constant pool of thrown exceptions.
      */
-    public final int[] getExceptionIndexTable() {
+    public int[] getExceptionIndexTable() {
         return exception_index_table;
     }
 
@@ -118,7 +118,7 @@ public final class ExceptionTable extends Attribute {
     /**
      * @return Length of exception table.
      */
-    public final int getNumberOfExceptions() {
+    public int getNumberOfExceptions() {
         return exception_index_table == null ? 0 : exception_index_table.length;
     }
 
@@ -126,7 +126,7 @@ public final class ExceptionTable extends Attribute {
     /**
      * @return class names of thrown exceptions
      */
-    public final String[] getExceptionNames() {
+    public String[] getExceptionNames() {
         final String[] names = new String[exception_index_table.length];
         for (int i = 0; i < exception_index_table.length; i++) {
             names[i] = super.getConstantPool().getConstantString(exception_index_table[i],
@@ -140,7 +140,7 @@ public final class ExceptionTable extends Attribute {
      * @param exception_index_table the list of exception indexes
      * Also redefines number_of_exceptions according to table length.
      */
-    public final void setExceptionIndexTable( final int[] exception_index_table ) {
+    public void setExceptionIndexTable( final int[] exception_index_table ) {
         this.exception_index_table = exception_index_table != null ? exception_index_table : new int[0];
     }
 
@@ -149,7 +149,7 @@ public final class ExceptionTable extends Attribute {
      * @return String representation, i.e., a list of thrown exceptions.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         String str;
         buf.append("Exceptions: ");
diff --git a/src/main/java/org/apache/bcel/classfile/Field.java b/src/main/java/org/apache/bcel/classfile/Field.java
index a04bd50..7dea3dd 100644
--- a/src/main/java/org/apache/bcel/classfile/Field.java
+++ b/src/main/java/org/apache/bcel/classfile/Field.java
@@ -99,7 +99,7 @@ public final class Field extends FieldOrMethod {
     /**
      * @return constant value associated with this field (may be null)
      */
-    public final ConstantValue getConstantValue() {
+    public ConstantValue getConstantValue() {
         for (final Attribute attribute : super.getAttributes()) {
             if (attribute.getTag() == Const.ATTR_CONSTANT_VALUE) {
                 return (ConstantValue) attribute;
@@ -116,7 +116,7 @@ public final class Field extends FieldOrMethod {
      * @return String representation of field, including the signature.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         String name;
         String signature;
         String access; // Short cuts to constant pool
@@ -144,7 +144,7 @@ public final class Field extends FieldOrMethod {
     /**
      * @return deep copy of this field
      */
-    public final Field copy( final ConstantPool _constant_pool ) {
+    public Field copy( final ConstantPool _constant_pool ) {
         return (Field) copy_(_constant_pool);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/InnerClass.java b/src/main/java/org/apache/bcel/classfile/InnerClass.java
index 94d2c88..f2404d4 100644
--- a/src/main/java/org/apache/bcel/classfile/InnerClass.java
+++ b/src/main/java/org/apache/bcel/classfile/InnerClass.java
@@ -92,7 +92,7 @@ public final class InnerClass implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(inner_class_index);
         file.writeShort(outer_class_index);
         file.writeShort(inner_name_index);
@@ -103,7 +103,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @return access flags of inner class.
      */
-    public final int getInnerAccessFlags() {
+    public int getInnerAccessFlags() {
         return inner_access_flags;
     }
 
@@ -111,7 +111,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @return class index of inner class.
      */
-    public final int getInnerClassIndex() {
+    public int getInnerClassIndex() {
         return inner_class_index;
     }
 
@@ -119,7 +119,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @return name index of inner class.
      */
-    public final int getInnerNameIndex() {
+    public int getInnerNameIndex() {
         return inner_name_index;
     }
 
@@ -127,7 +127,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @return class index of outer class.
      */
-    public final int getOuterClassIndex() {
+    public int getOuterClassIndex() {
         return outer_class_index;
     }
 
@@ -135,7 +135,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @param inner_access_flags access flags for this inner class
      */
-    public final void setInnerAccessFlags( final int inner_access_flags ) {
+    public void setInnerAccessFlags( final int inner_access_flags ) {
         this.inner_access_flags = inner_access_flags;
     }
 
@@ -143,7 +143,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @param inner_class_index index into the constant pool for this class
      */
-    public final void setInnerClassIndex( final int inner_class_index ) {
+    public void setInnerClassIndex( final int inner_class_index ) {
         this.inner_class_index = inner_class_index;
     }
 
@@ -151,7 +151,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @param inner_name_index index into the constant pool for this class's name
      */
-    public final void setInnerNameIndex( final int inner_name_index ) { // TODO unused
+    public void setInnerNameIndex( final int inner_name_index ) { // TODO unused
         this.inner_name_index = inner_name_index;
     }
 
@@ -159,7 +159,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @param outer_class_index index into the constant pool for the owning class
      */
-    public final void setOuterClassIndex( final int outer_class_index ) { // TODO unused
+    public void setOuterClassIndex( final int outer_class_index ) { // TODO unused
         this.outer_class_index = outer_class_index;
     }
 
@@ -168,7 +168,7 @@ public final class InnerClass implements Cloneable, Node {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "InnerClass(" + inner_class_index + ", " + outer_class_index + ", "
                 + inner_name_index + ", " + inner_access_flags + ")";
     }
@@ -177,7 +177,7 @@ public final class InnerClass implements Cloneable, Node {
     /**
      * @return Resolved string representation
      */
-    public final String toString( final ConstantPool constant_pool ) {
+    public String toString( final ConstantPool constant_pool ) {
         String outer_class_name;
         String inner_name;
         String inner_class_name = constant_pool.getConstantString(inner_class_index,
diff --git a/src/main/java/org/apache/bcel/classfile/InnerClasses.java b/src/main/java/org/apache/bcel/classfile/InnerClasses.java
index 565294f..0158462 100644
--- a/src/main/java/org/apache/bcel/classfile/InnerClasses.java
+++ b/src/main/java/org/apache/bcel/classfile/InnerClasses.java
@@ -98,7 +98,7 @@ public final class InnerClasses extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(inner_classes.length);
         for (final InnerClass inner_class : inner_classes) {
@@ -110,7 +110,7 @@ public final class InnerClasses extends Attribute {
     /**
      * @return array of inner class "records"
      */
-    public final InnerClass[] getInnerClasses() {
+    public InnerClass[] getInnerClasses() {
         return inner_classes;
     }
 
@@ -118,7 +118,7 @@ public final class InnerClasses extends Attribute {
     /**
      * @param inner_classes the array of inner classes
      */
-    public final void setInnerClasses( final InnerClass[] inner_classes ) {
+    public void setInnerClasses( final InnerClass[] inner_classes ) {
         this.inner_classes = inner_classes != null ? inner_classes : new InnerClass[0];
     }
 
@@ -127,7 +127,7 @@ public final class InnerClasses extends Attribute {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         buf.append("InnerClasses(");
         buf.append(inner_classes.length);
diff --git a/src/main/java/org/apache/bcel/classfile/LineNumber.java b/src/main/java/org/apache/bcel/classfile/LineNumber.java
index 0ec2f1b..75b9d51 100644
--- a/src/main/java/org/apache/bcel/classfile/LineNumber.java
+++ b/src/main/java/org/apache/bcel/classfile/LineNumber.java
@@ -86,7 +86,7 @@ public final class LineNumber implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException if an I/O Exception occurs in writeShort
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(start_pc);
         file.writeShort(line_number);
     }
@@ -95,7 +95,7 @@ public final class LineNumber implements Cloneable, Node {
     /**
      * @return Corresponding source line
      */
-    public final int getLineNumber() {
+    public int getLineNumber() {
         return 0xffff & line_number;
     }
 
@@ -103,7 +103,7 @@ public final class LineNumber implements Cloneable, Node {
     /**
      * @return PC in code
      */
-    public final int getStartPC() {
+    public int getStartPC() {
         return  0xffff & start_pc;
     }
 
@@ -111,7 +111,7 @@ public final class LineNumber implements Cloneable, Node {
     /**
      * @param line_number the source line number
      */
-    public final void setLineNumber( final int line_number ) {
+    public void setLineNumber( final int line_number ) {
         this.line_number = (short) line_number;
     }
 
@@ -119,7 +119,7 @@ public final class LineNumber implements Cloneable, Node {
     /**
      * @param start_pc the pc for this line number
      */
-    public final void setStartPC( final int start_pc ) {
+    public void setStartPC( final int start_pc ) {
         this.start_pc = (short) start_pc;
     }
 
@@ -128,7 +128,7 @@ public final class LineNumber implements Cloneable, Node {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "LineNumber(" + start_pc + ", " + line_number + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/LineNumberTable.java b/src/main/java/org/apache/bcel/classfile/LineNumberTable.java
index 7279b63..bca93cf 100644
--- a/src/main/java/org/apache/bcel/classfile/LineNumberTable.java
+++ b/src/main/java/org/apache/bcel/classfile/LineNumberTable.java
@@ -98,7 +98,7 @@ public final class LineNumberTable extends Attribute {
      * @throws IOException if an I/O Exception occurs in writeShort
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(line_number_table.length);
         for (final LineNumber lineNumber : line_number_table) {
@@ -110,7 +110,7 @@ public final class LineNumberTable extends Attribute {
     /**
      * @return Array of (pc offset, line number) pairs.
      */
-    public final LineNumber[] getLineNumberTable() {
+    public LineNumber[] getLineNumberTable() {
         return line_number_table;
     }
 
@@ -118,7 +118,7 @@ public final class LineNumberTable extends Attribute {
     /**
      * @param line_number_table the line number entries for this table
      */
-    public final void setLineNumberTable( final LineNumber[] line_number_table ) {
+    public void setLineNumberTable( final LineNumber[] line_number_table ) {
         this.line_number_table = line_number_table;
     }
 
@@ -127,7 +127,7 @@ public final class LineNumberTable extends Attribute {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         final StringBuilder line = new StringBuilder();
         final String newLine = System.getProperty("line.separator", "\n");
@@ -209,7 +209,7 @@ public final class LineNumberTable extends Attribute {
     }
 
 
-    public final int getTableLength() {
+    public int getTableLength() {
         return line_number_table == null ? 0 : line_number_table.length;
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/LocalVariable.java b/src/main/java/org/apache/bcel/classfile/LocalVariable.java
index 5c8fa72..f8100cf 100644
--- a/src/main/java/org/apache/bcel/classfile/LocalVariable.java
+++ b/src/main/java/org/apache/bcel/classfile/LocalVariable.java
@@ -134,7 +134,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
      * @param file Output file stream
      * @throws IOException
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(start_pc);
         file.writeShort(length);
         file.writeShort(name_index);
@@ -146,7 +146,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Constant pool used by this object.
      */
-    public final ConstantPool getConstantPool() {
+    public ConstantPool getConstantPool() {
         return constant_pool;
     }
 
@@ -154,7 +154,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Variable is valid within getStartPC() .. getStartPC()+getLength()
      */
-    public final int getLength() {
+    public int getLength() {
         return length;
     }
 
@@ -162,7 +162,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Variable name.
      */
-    public final String getName() {
+    public String getName() {
         ConstantUtf8 c;
         c = (ConstantUtf8) constant_pool.getConstant(name_index, Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -172,7 +172,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Index in constant pool of variable name.
      */
-    public final int getNameIndex() {
+    public int getNameIndex() {
         return name_index;
     }
 
@@ -180,7 +180,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Signature.
      */
-    public final String getSignature() {
+    public String getSignature() {
         ConstantUtf8 c;
         c = (ConstantUtf8) constant_pool.getConstant(signature_index, Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -190,7 +190,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Index in constant pool of variable signature.
      */
-    public final int getSignatureIndex() {
+    public int getSignatureIndex() {
         return signature_index;
     }
 
@@ -198,7 +198,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return index of register where variable is stored
      */
-    public final int getIndex() {
+    public int getIndex() {
         return index;
     }
 
@@ -206,7 +206,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return index of register where variable was originally stored
      */
-    public final int getOrigIndex() {
+    public int getOrigIndex() {
         return orig_index;
     }
 
@@ -214,7 +214,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @return Start of range where he variable is valid
      */
-    public final int getStartPC() {
+    public int getStartPC() {
         return start_pc;
     }
 
@@ -222,7 +222,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /*
      * Helper method shared with LocalVariableTypeTable
      */
-    final String toStringShared( final boolean typeTable ) {
+    String toStringShared( final boolean typeTable ) {
         final String name = getName();
         final String signature = Utility.signatureToString(getSignature(), false);
         final String label = "LocalVariable" + (typeTable ? "Types" : "" );
@@ -234,7 +234,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param constant_pool Constant pool to be used for this object.
      */
-    public final void setConstantPool( final ConstantPool constant_pool ) {
+    public void setConstantPool( final ConstantPool constant_pool ) {
         this.constant_pool = constant_pool;
     }
 
@@ -242,7 +242,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param length the length of this local variable
      */
-    public final void setLength( final int length ) {
+    public void setLength( final int length ) {
         this.length = length;
     }
 
@@ -250,7 +250,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param name_index the index into the constant pool for the name of this variable
      */
-    public final void setNameIndex( final int name_index ) { // TODO unused
+    public void setNameIndex( final int name_index ) { // TODO unused
         this.name_index = name_index;
     }
 
@@ -258,7 +258,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param signature_index the index into the constant pool for the signature of this variable
      */
-    public final void setSignatureIndex( final int signature_index ) { // TODO unused
+    public void setSignatureIndex( final int signature_index ) { // TODO unused
         this.signature_index = signature_index;
     }
 
@@ -266,7 +266,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param index the index in the local variable table of this variable
      */
-    public final void setIndex( final int index ) { // TODO unused
+    public void setIndex( final int index ) { // TODO unused
         this.index = index;
     }
 
@@ -274,7 +274,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
     /**
      * @param start_pc Specify range where the local variable is valid.
      */
-    public final void setStartPC( final int start_pc ) { // TODO unused
+    public void setStartPC( final int start_pc ) { // TODO unused
         this.start_pc = start_pc;
     }
 
@@ -283,7 +283,7 @@ public final class LocalVariable implements Cloneable, Node, Constants {
      * @return string representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return toStringShared(false);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/Method.java b/src/main/java/org/apache/bcel/classfile/Method.java
index c48b8e1..8f9b8d6 100644
--- a/src/main/java/org/apache/bcel/classfile/Method.java
+++ b/src/main/java/org/apache/bcel/classfile/Method.java
@@ -112,7 +112,7 @@ public final class Method extends FieldOrMethod {
     /**
      * @return Code attribute of method, if any
      */
-    public final Code getCode() {
+    public Code getCode() {
         for (final Attribute attribute : super.getAttributes()) {
             if (attribute instanceof Code) {
                 return (Code) attribute;
@@ -126,7 +126,7 @@ public final class Method extends FieldOrMethod {
      * @return ExceptionTable attribute of method, if any, i.e., list all
      * exceptions the method may throw not exception handlers!
      */
-    public final ExceptionTable getExceptionTable() {
+    public ExceptionTable getExceptionTable() {
         for (final Attribute attribute : super.getAttributes()) {
             if (attribute instanceof ExceptionTable) {
                 return (ExceptionTable) attribute;
@@ -139,7 +139,7 @@ public final class Method extends FieldOrMethod {
     /** @return LocalVariableTable of code attribute if any, i.e. the call is forwarded
      * to the Code atribute.
      */
-    public final LocalVariableTable getLocalVariableTable() {
+    public LocalVariableTable getLocalVariableTable() {
         final Code code = getCode();
         if (code == null) {
             return null;
@@ -151,7 +151,7 @@ public final class Method extends FieldOrMethod {
     /** @return LineNumberTable of code attribute if any, i.e. the call is forwarded
      * to the Code atribute.
      */
-    public final LineNumberTable getLineNumberTable() {
+    public LineNumberTable getLineNumberTable() {
         final Code code = getCode();
         if (code == null) {
             return null;
@@ -167,7 +167,7 @@ public final class Method extends FieldOrMethod {
      * @return String representation of the method.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final String access = Utility.accessToString(super.getAccessFlags());
         // Get name and signature from constant pool
         ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(super.getSignatureIndex(), Const.CONSTANT_Utf8);
@@ -196,7 +196,7 @@ public final class Method extends FieldOrMethod {
     /**
      * @return deep copy of this method
      */
-    public final Method copy( final ConstantPool _constant_pool ) {
+    public Method copy( final ConstantPool _constant_pool ) {
         return (Method) copy_(_constant_pool);
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/Module.java b/src/main/java/org/apache/bcel/classfile/Module.java
index 824cd8d..a0a9d62 100644
--- a/src/main/java/org/apache/bcel/classfile/Module.java
+++ b/src/main/java/org/apache/bcel/classfile/Module.java
@@ -108,7 +108,7 @@ public final class Module extends Attribute {
      * @return table of required modules
      * @see ModuleRequires
      */
-    public final ModuleRequires[] getRequiresTable() {
+    public ModuleRequires[] getRequiresTable() {
         return requires_table;
     }
 
@@ -117,7 +117,7 @@ public final class Module extends Attribute {
      * @return table of exported interfaces
      * @see ModuleExports
      */
-    public final ModuleExports[] getExportsTable() {
+    public ModuleExports[] getExportsTable() {
         return exports_table;
     }
 
@@ -126,7 +126,7 @@ public final class Module extends Attribute {
      * @return table of provided interfaces
      * @see ModuleOpens
      */
-    public final ModuleOpens[] getOpensTable() {
+    public ModuleOpens[] getOpensTable() {
         return opens_table;
     }
 
@@ -135,7 +135,7 @@ public final class Module extends Attribute {
      * @return table of provided interfaces
      * @see ModuleProvides
      */
-    public final ModuleProvides[] getProvidesTable() {
+    public ModuleProvides[] getProvidesTable() {
         return provides_table;
     }
 
@@ -147,7 +147,7 @@ public final class Module extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
 
         file.writeShort(module_name_index);
@@ -185,7 +185,7 @@ public final class Module extends Attribute {
      * @return String representation, i.e., a list of packages.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final ConstantPool cp = super.getConstantPool();
         final StringBuilder buf = new StringBuilder();
         buf.append("Module:\n");
diff --git a/src/main/java/org/apache/bcel/classfile/ModuleExports.java b/src/main/java/org/apache/bcel/classfile/ModuleExports.java
index 51c0399..e654b86 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleExports.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleExports.java
@@ -75,7 +75,7 @@ public final class ModuleExports implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException if an I/O Exception occurs in writeShort
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(exports_index);
         file.writeShort(exports_flags);
         file.writeShort(exports_to_count);
@@ -89,7 +89,7 @@ public final class ModuleExports implements Cloneable, Node {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "exports(" + exports_index + ", " + exports_flags + ", " + exports_to_count + ", ...)";
     }
 
@@ -97,7 +97,7 @@ public final class ModuleExports implements Cloneable, Node {
     /**
      * @return Resolved string representation
      */
-    public final String toString( final ConstantPool constant_pool ) {
+    public String toString( final ConstantPool constant_pool ) {
         final StringBuilder buf = new StringBuilder();
         final String package_name = constant_pool.constantToString(exports_index, Const.CONSTANT_Package);
         buf.append(Utility.compactClassName(package_name, false));
diff --git a/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java b/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
index f270f05..2de3582 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
@@ -90,7 +90,7 @@ public final class ModuleMainClass extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(main_class_index);
     }
@@ -99,7 +99,7 @@ public final class ModuleMainClass extends Attribute {
     /**
      * @return index into constant pool of host class name.
      */
-    public final int getHostClassIndex() {
+    public int getHostClassIndex() {
         return main_class_index;
     }
 
@@ -107,7 +107,7 @@ public final class ModuleMainClass extends Attribute {
     /**
      * @param int the host class index
      */
-    public final void setHostClassIndex( final int main_class_index ) {
+    public void setHostClassIndex( final int main_class_index ) {
         this.main_class_index = main_class_index;
     }
 
@@ -116,7 +116,7 @@ public final class ModuleMainClass extends Attribute {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         buf.append("ModuleMainClass: ");
         final String class_name = super.getConstantPool().getConstantString(main_class_index, Const.CONSTANT_Class);
diff --git a/src/main/java/org/apache/bcel/classfile/ModuleOpens.java b/src/main/java/org/apache/bcel/classfile/ModuleOpens.java
index 9539afc..a4cf6da 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleOpens.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleOpens.java
@@ -75,7 +75,7 @@ public final class ModuleOpens implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException if an I/O Exception occurs in writeShort
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(opens_index);
         file.writeShort(opens_flags);
         file.writeShort(opens_to_count);
@@ -89,7 +89,7 @@ public final class ModuleOpens implements Cloneable, Node {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "opens(" + opens_index + ", " + opens_flags + ", " + opens_to_count + ", ...)";
     }
 
@@ -97,7 +97,7 @@ public final class ModuleOpens implements Cloneable, Node {
     /**
      * @return Resolved string representation
      */
-    public final String toString( final ConstantPool constant_pool ) {
+    public String toString( final ConstantPool constant_pool ) {
         final StringBuilder buf = new StringBuilder();
         final String package_name = constant_pool.constantToString(opens_index, Const.CONSTANT_Package);
         buf.append(Utility.compactClassName(package_name, false));
diff --git a/src/main/java/org/apache/bcel/classfile/ModulePackages.java b/src/main/java/org/apache/bcel/classfile/ModulePackages.java
index 95df9dc..b7ac547 100644
--- a/src/main/java/org/apache/bcel/classfile/ModulePackages.java
+++ b/src/main/java/org/apache/bcel/classfile/ModulePackages.java
@@ -94,7 +94,7 @@ public final class ModulePackages extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(package_index_table.length);
         for (final int index : package_index_table) {
@@ -106,7 +106,7 @@ public final class ModulePackages extends Attribute {
     /**
      * @return array of indices into constant pool of package names.
      */
-    public final int[] getPackageIndexTable() {
+    public int[] getPackageIndexTable() {
         return package_index_table;
     }
 
@@ -114,7 +114,7 @@ public final class ModulePackages extends Attribute {
     /**
      * @return Length of package table.
      */
-    public final int getNumberOfPackages() {
+    public int getNumberOfPackages() {
         return package_index_table == null ? 0 : package_index_table.length;
     }
 
@@ -122,7 +122,7 @@ public final class ModulePackages extends Attribute {
     /**
      * @return string array of package names
      */
-    public final String[] getPackageNames() {
+    public String[] getPackageNames() {
         final String[] names = new String[package_index_table.length];
         for (int i = 0; i < package_index_table.length; i++) {
             names[i] = super.getConstantPool().getConstantString(package_index_table[i],
@@ -136,7 +136,7 @@ public final class ModulePackages extends Attribute {
      * @param package_index_table the list of package indexes
      * Also redefines number_of_packages according to table length.
      */
-    public final void setPackageIndexTable( final int[] package_index_table ) {
+    public void setPackageIndexTable( final int[] package_index_table ) {
         this.package_index_table = package_index_table != null ? package_index_table : new int[0];
     }
 
@@ -145,7 +145,7 @@ public final class ModulePackages extends Attribute {
      * @return String representation, i.e., a list of packages.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         buf.append("ModulePackages(");
         buf.append(package_index_table.length);
diff --git a/src/main/java/org/apache/bcel/classfile/ModuleProvides.java b/src/main/java/org/apache/bcel/classfile/ModuleProvides.java
index a98834e..5996e6b 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleProvides.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleProvides.java
@@ -73,7 +73,7 @@ public final class ModuleProvides implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException if an I/O Exception occurs in writeShort
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(provides_index);
         file.writeShort(provides_with_count);
         for (final int entry : provides_with_index) {
@@ -86,7 +86,7 @@ public final class ModuleProvides implements Cloneable, Node {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "provides(" + provides_index + ", " + provides_with_count + ", ...)";
     }
 
@@ -94,7 +94,7 @@ public final class ModuleProvides implements Cloneable, Node {
     /**
      * @return Resolved string representation
      */
-    public final String toString( final ConstantPool constant_pool ) {
+    public String toString( final ConstantPool constant_pool ) {
         final StringBuilder buf = new StringBuilder();
         final String interface_name = constant_pool.constantToString(provides_index, Const.CONSTANT_Class);
         buf.append(Utility.compactClassName(interface_name, false));
diff --git a/src/main/java/org/apache/bcel/classfile/ModuleRequires.java b/src/main/java/org/apache/bcel/classfile/ModuleRequires.java
index 4f24391..136a227 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleRequires.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleRequires.java
@@ -70,7 +70,7 @@ public final class ModuleRequires implements Cloneable, Node {
      * @param file Output file stream
      * @throws IOException if an I/O Exception occurs in writeShort
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeShort(requires_index);
         file.writeShort(requires_flags);
         file.writeShort(requires_version_index);
@@ -81,7 +81,7 @@ public final class ModuleRequires implements Cloneable, Node {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "requires(" + requires_index + ", " + String.format("%04x", requires_flags) + ", " + requires_version_index + ")";
     }
 
@@ -89,7 +89,7 @@ public final class ModuleRequires implements Cloneable, Node {
     /**
      * @return Resolved string representation
      */
-    public final String toString( final ConstantPool constant_pool ) {
+    public String toString( final ConstantPool constant_pool ) {
         final StringBuilder buf = new StringBuilder();
         final String module_name = constant_pool.constantToString(requires_index, Const.CONSTANT_Module);
         buf.append(Utility.compactClassName(module_name, false));
diff --git a/src/main/java/org/apache/bcel/classfile/NestHost.java b/src/main/java/org/apache/bcel/classfile/NestHost.java
index 44843c2..68fdfcb 100644
--- a/src/main/java/org/apache/bcel/classfile/NestHost.java
+++ b/src/main/java/org/apache/bcel/classfile/NestHost.java
@@ -91,7 +91,7 @@ public final class NestHost extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(host_class_index);
     }
@@ -100,7 +100,7 @@ public final class NestHost extends Attribute {
     /**
      * @return index into constant pool of host class name.
      */
-    public final int getHostClassIndex() {
+    public int getHostClassIndex() {
         return host_class_index;
     }
 
@@ -108,7 +108,7 @@ public final class NestHost extends Attribute {
     /**
      * @param int the host class index
      */
-    public final void setHostClassIndex( final int host_class_index ) {
+    public void setHostClassIndex( final int host_class_index ) {
         this.host_class_index = host_class_index;
     }
 
@@ -117,7 +117,7 @@ public final class NestHost extends Attribute {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         buf.append("NestHost: ");
         final String class_name = super.getConstantPool().getConstantString(host_class_index, Const.CONSTANT_Class);
diff --git a/src/main/java/org/apache/bcel/classfile/NestMembers.java b/src/main/java/org/apache/bcel/classfile/NestMembers.java
index da9b6b4..4dbb185 100644
--- a/src/main/java/org/apache/bcel/classfile/NestMembers.java
+++ b/src/main/java/org/apache/bcel/classfile/NestMembers.java
@@ -95,7 +95,7 @@ public final class NestMembers extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(classes.length);
         for (final int index : classes) {
@@ -107,7 +107,7 @@ public final class NestMembers extends Attribute {
     /**
      * @return array of indices into constant pool of class names.
      */
-    public final int[] getClasses() {
+    public int[] getClasses() {
         return classes;
     }
 
@@ -115,7 +115,7 @@ public final class NestMembers extends Attribute {
     /**
      * @return Length of classes table.
      */
-    public final int getNumberClasses() {
+    public int getNumberClasses() {
         return classes == null ? 0 : classes.length;
     }
 
@@ -123,7 +123,7 @@ public final class NestMembers extends Attribute {
     /**
      * @return string array of class names
      */
-    public final String[] getClassNames() {
+    public String[] getClassNames() {
         final String[] names = new String[classes.length];
         for (int i = 0; i < classes.length; i++) {
             names[i] = super.getConstantPool().getConstantString(classes[i],
@@ -137,7 +137,7 @@ public final class NestMembers extends Attribute {
      * @param classes the list of class indexes
      * Also redefines number_of_classes according to table length.
      */
-    public final void setClasses( final int[] classes ) {
+    public void setClasses( final int[] classes ) {
         this.classes = classes != null ? classes : new int[0];
     }
 
@@ -146,7 +146,7 @@ public final class NestMembers extends Attribute {
      * @return String representation, i.e., a list of classes.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder();
         buf.append("NestMembers(");
         buf.append(classes.length);
diff --git a/src/main/java/org/apache/bcel/classfile/PMGClass.java b/src/main/java/org/apache/bcel/classfile/PMGClass.java
index 4f33a24..19f0d1d 100644
--- a/src/main/java/org/apache/bcel/classfile/PMGClass.java
+++ b/src/main/java/org/apache/bcel/classfile/PMGClass.java
@@ -94,7 +94,7 @@ public final class PMGClass extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(pmg_index);
         file.writeShort(pmg_class_index);
@@ -104,7 +104,7 @@ public final class PMGClass extends Attribute {
     /**
      * @return Index in constant pool of source file name.
      */
-    public final int getPMGClassIndex() {
+    public int getPMGClassIndex() {
         return pmg_class_index;
     }
 
@@ -112,7 +112,7 @@ public final class PMGClass extends Attribute {
     /**
      * @param pmg_class_index
      */
-    public final void setPMGClassIndex( final int pmg_class_index ) {
+    public void setPMGClassIndex( final int pmg_class_index ) {
         this.pmg_class_index = pmg_class_index;
     }
 
@@ -120,7 +120,7 @@ public final class PMGClass extends Attribute {
     /**
      * @return Index in constant pool of source file name.
      */
-    public final int getPMGIndex() {
+    public int getPMGIndex() {
         return pmg_index;
     }
 
@@ -128,7 +128,7 @@ public final class PMGClass extends Attribute {
     /**
      * @param pmg_index
      */
-    public final void setPMGIndex( final int pmg_index ) {
+    public void setPMGIndex( final int pmg_index ) {
         this.pmg_index = pmg_index;
     }
 
@@ -136,7 +136,7 @@ public final class PMGClass extends Attribute {
     /**
      * @return PMG name.
      */
-    public final String getPMGName() {
+    public String getPMGName() {
         final ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(pmg_index,
                 Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -146,7 +146,7 @@ public final class PMGClass extends Attribute {
     /**
      * @return PMG class name.
      */
-    public final String getPMGClassName() {
+    public String getPMGClassName() {
         final ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(pmg_class_index,
                 Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -157,7 +157,7 @@ public final class PMGClass extends Attribute {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "PMGClass(" + getPMGName() + ", " + getPMGClassName() + ")";
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/Signature.java b/src/main/java/org/apache/bcel/classfile/Signature.java
index 217640f..38d9e6d 100644
--- a/src/main/java/org/apache/bcel/classfile/Signature.java
+++ b/src/main/java/org/apache/bcel/classfile/Signature.java
@@ -91,7 +91,7 @@ public final class Signature extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(signature_index);
     }
@@ -100,7 +100,7 @@ public final class Signature extends Attribute {
     /**
      * @return Index in constant pool of source file name.
      */
-    public final int getSignatureIndex() {
+    public int getSignatureIndex() {
         return signature_index;
     }
 
@@ -108,7 +108,7 @@ public final class Signature extends Attribute {
     /**
      * @param signature_index the index info the constant pool of this signature
      */
-    public final void setSignatureIndex( final int signature_index ) {
+    public void setSignatureIndex( final int signature_index ) {
         this.signature_index = signature_index;
     }
 
@@ -116,7 +116,7 @@ public final class Signature extends Attribute {
     /**
      * @return GJ signature.
      */
-    public final String getSignature() {
+    public String getSignature() {
         final ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(signature_index,
                 Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -132,12 +132,12 @@ public final class Signature extends Attribute {
         }
 
 
-        final String getData() {
+        String getData() {
             return new String(buf);
         }
 
 
-        final void unread() {
+        void unread() {
             if (pos > 0) {
                 pos--;
             }
@@ -253,7 +253,7 @@ public final class Signature extends Attribute {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final String s = getSignature();
         return "Signature: " + s;
     }
diff --git a/src/main/java/org/apache/bcel/classfile/SourceFile.java b/src/main/java/org/apache/bcel/classfile/SourceFile.java
index eba2709..7688ac1 100644
--- a/src/main/java/org/apache/bcel/classfile/SourceFile.java
+++ b/src/main/java/org/apache/bcel/classfile/SourceFile.java
@@ -98,7 +98,7 @@ public final class SourceFile extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(sourcefile_index);
     }
@@ -107,7 +107,7 @@ public final class SourceFile extends Attribute {
     /**
      * @return Index in constant pool of source file name.
      */
-    public final int getSourceFileIndex() {
+    public int getSourceFileIndex() {
         return sourcefile_index;
     }
 
@@ -115,7 +115,7 @@ public final class SourceFile extends Attribute {
     /**
      * @param sourcefile_index
      */
-    public final void setSourceFileIndex( final int sourcefile_index ) {
+    public void setSourceFileIndex( final int sourcefile_index ) {
         this.sourcefile_index = sourcefile_index;
     }
 
@@ -123,7 +123,7 @@ public final class SourceFile extends Attribute {
     /**
      * @return Source file name.
      */
-    public final String getSourceFileName() {
+    public String getSourceFileName() {
         final ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(sourcefile_index,
                 Const.CONSTANT_Utf8);
         return c.getBytes();
@@ -134,7 +134,7 @@ public final class SourceFile extends Attribute {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "SourceFile: " + getSourceFileName();
     }
 
diff --git a/src/main/java/org/apache/bcel/classfile/StackMap.java b/src/main/java/org/apache/bcel/classfile/StackMap.java
index b62f33c..99ec3b2 100644
--- a/src/main/java/org/apache/bcel/classfile/StackMap.java
+++ b/src/main/java/org/apache/bcel/classfile/StackMap.java
@@ -79,7 +79,7 @@ public final class StackMap extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         file.writeShort(map.length);
         for (final StackMapEntry entry : map) {
@@ -91,7 +91,7 @@ public final class StackMap extends Attribute {
     /**
      * @return Array of stack map entries
      */
-    public final StackMapEntry[] getStackMap() {
+    public StackMapEntry[] getStackMap() {
         return map;
     }
 
@@ -99,7 +99,7 @@ public final class StackMap extends Attribute {
     /**
      * @param map Array of stack map entries
      */
-    public final void setStackMap( final StackMapEntry[] map ) {
+    public void setStackMap( final StackMapEntry[] map ) {
         this.map = map;
         int len = 2; // Length of 'number_of_entries' field prior to the array of stack maps
         for (final StackMapEntry element : map) {
@@ -113,7 +113,7 @@ public final class StackMap extends Attribute {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder("StackMap(");
         for (int i = 0; i < map.length; i++) {
             buf.append(map[i]);
@@ -154,7 +154,7 @@ public final class StackMap extends Attribute {
     }
 
 
-    public final int getMapLength() {
+    public int getMapLength() {
         return map == null ? 0 : map.length;
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/StackMapEntry.java b/src/main/java/org/apache/bcel/classfile/StackMapEntry.java
index b9e1468..4f4f97a 100644
--- a/src/main/java/org/apache/bcel/classfile/StackMapEntry.java
+++ b/src/main/java/org/apache/bcel/classfile/StackMapEntry.java
@@ -137,7 +137,7 @@ public final class StackMapEntry implements Node, Cloneable
      * @param file Output file stream
      * @throws IOException
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.write(frame_type);
         if (frame_type >= Const.SAME_FRAME && frame_type <= Const.SAME_FRAME_MAX) {
             // nothing to be done
@@ -177,7 +177,7 @@ public final class StackMapEntry implements Node, Cloneable
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder(64);
         buf.append("(");
         if (frame_type >= Const.SAME_FRAME && frame_type <= Const.SAME_FRAME_MAX) {
@@ -421,7 +421,7 @@ public final class StackMapEntry implements Node, Cloneable
     /**
      * @return Constant pool used by this object.
      */
-    public final ConstantPool getConstantPool() {
+    public ConstantPool getConstantPool() {
         return constant_pool;
     }
 
@@ -429,7 +429,7 @@ public final class StackMapEntry implements Node, Cloneable
     /**
      * @param constant_pool Constant pool to be used for this object.
      */
-    public final void setConstantPool( final ConstantPool constant_pool ) {
+    public void setConstantPool( final ConstantPool constant_pool ) {
         this.constant_pool = constant_pool;
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/StackMapType.java b/src/main/java/org/apache/bcel/classfile/StackMapType.java
index cf5d29f..4e35cb2 100644
--- a/src/main/java/org/apache/bcel/classfile/StackMapType.java
+++ b/src/main/java/org/apache/bcel/classfile/StackMapType.java
@@ -98,7 +98,7 @@ public final class StackMapType implements Cloneable {
      * @param file Output file stream
      * @throws IOException
      */
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         file.writeByte(type);
         if (hasIndex()) {
             file.writeShort(getIndex());
@@ -108,7 +108,7 @@ public final class StackMapType implements Cloneable {
 
     /** @return true, if type is either ITEM_Object or ITEM_NewObject
      */
-    public final boolean hasIndex() {
+    public boolean hasIndex() {
         return type == Const.ITEM_Object || type == Const.ITEM_NewObject;
     }
 
@@ -131,7 +131,7 @@ public final class StackMapType implements Cloneable {
      * @return String representation
      */
     @Override
-    public final String toString() {
+    public String toString() {
         return "(type=" + Const.getItemName(type) + printIndex() + ")";
     }
 
@@ -152,7 +152,7 @@ public final class StackMapType implements Cloneable {
     /**
      * @return Constant pool used by this object.
      */
-    public final ConstantPool getConstantPool() {
+    public ConstantPool getConstantPool() {
         return constant_pool;
     }
 
@@ -160,7 +160,7 @@ public final class StackMapType implements Cloneable {
     /**
      * @param constant_pool Constant pool to be used for this object.
      */
-    public final void setConstantPool( final ConstantPool constant_pool ) {
+    public void setConstantPool( final ConstantPool constant_pool ) {
         this.constant_pool = constant_pool;
     }
 }
diff --git a/src/main/java/org/apache/bcel/classfile/Synthetic.java b/src/main/java/org/apache/bcel/classfile/Synthetic.java
index f952dea..af24ecd 100644
--- a/src/main/java/org/apache/bcel/classfile/Synthetic.java
+++ b/src/main/java/org/apache/bcel/classfile/Synthetic.java
@@ -102,7 +102,7 @@ public final class Synthetic extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         if (super.getLength() > 0) {
             file.write(bytes, 0, super.getLength());
@@ -113,7 +113,7 @@ public final class Synthetic extends Attribute {
     /**
      * @return data bytes.
      */
-    public final byte[] getBytes() {
+    public byte[] getBytes() {
         return bytes;
     }
 
@@ -121,7 +121,7 @@ public final class Synthetic extends Attribute {
     /**
      * @param bytes
      */
-    public final void setBytes( final byte[] bytes ) {
+    public void setBytes( final byte[] bytes ) {
         this.bytes = bytes;
     }
 
@@ -130,7 +130,7 @@ public final class Synthetic extends Attribute {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         final StringBuilder buf = new StringBuilder("Synthetic");
         if (super.getLength() > 0) {
             buf.append(" ").append(Utility.toHexString(bytes));
diff --git a/src/main/java/org/apache/bcel/classfile/Unknown.java b/src/main/java/org/apache/bcel/classfile/Unknown.java
index 29a65a7..70a6f64 100644
--- a/src/main/java/org/apache/bcel/classfile/Unknown.java
+++ b/src/main/java/org/apache/bcel/classfile/Unknown.java
@@ -119,7 +119,7 @@ public final class Unknown extends Attribute {
      * @throws IOException
      */
     @Override
-    public final void dump( final DataOutputStream file ) throws IOException {
+    public void dump( final DataOutputStream file ) throws IOException {
         super.dump(file);
         if (super.getLength() > 0) {
             file.write(bytes, 0, super.getLength());
@@ -130,7 +130,7 @@ public final class Unknown extends Attribute {
     /**
      * @return data bytes.
      */
-    public final byte[] getBytes() {
+    public byte[] getBytes() {
         return bytes;
     }
 
@@ -139,7 +139,7 @@ public final class Unknown extends Attribute {
      * @return name of attribute.
      */
     @Override
-    public final String getName() {
+    public String getName() {
         return name;
     }
 
@@ -147,7 +147,7 @@ public final class Unknown extends Attribute {
     /**
      * @param bytes the bytes to set
      */
-    public final void setBytes( final byte[] bytes ) {
+    public void setBytes( final byte[] bytes ) {
         this.bytes = bytes;
     }
 
@@ -156,7 +156,7 @@ public final class Unknown extends Attribute {
      * @return String representation.
      */
     @Override
-    public final String toString() {
+    public String toString() {
         if (super.getLength() == 0 || bytes == null) {
             return "(Unknown attribute " + name + ")";
         }
diff --git a/src/main/java/org/apache/bcel/generic/PUSH.java b/src/main/java/org/apache/bcel/generic/PUSH.java
index 536be28..f652da9 100644
--- a/src/main/java/org/apache/bcel/generic/PUSH.java
+++ b/src/main/java/org/apache/bcel/generic/PUSH.java
@@ -171,12 +171,12 @@ public final class PUSH implements CompoundInstruction, VariableLengthInstructio
 
 
     @Override
-    public final InstructionList getInstructionList() {
+    public InstructionList getInstructionList() {
         return new InstructionList(instruction);
     }
 
 
-    public final Instruction getInstruction() {
+    public Instruction getInstruction() {
         return instruction;
     }
 
diff --git a/src/main/java/org/apache/bcel/generic/SWITCH.java b/src/main/java/org/apache/bcel/generic/SWITCH.java
index 0e094e0..390b3b9 100644
--- a/src/main/java/org/apache/bcel/generic/SWITCH.java
+++ b/src/main/java/org/apache/bcel/generic/SWITCH.java
@@ -144,12 +144,12 @@ public final class SWITCH implements CompoundInstruction {
 
 
     @Override
-    public final InstructionList getInstructionList() {
+    public InstructionList getInstructionList() {
         return new InstructionList(instruction);
     }
 
 
-    public final Instruction getInstruction() {
+    public Instruction getInstruction() {
         return instruction;
     }
 }
diff --git a/src/main/java/org/apache/bcel/util/AttributeHTML.java b/src/main/java/org/apache/bcel/util/AttributeHTML.java
index 4698705..dfe5c8d 100644
--- a/src/main/java/org/apache/bcel/util/AttributeHTML.java
+++ b/src/main/java/org/apache/bcel/util/AttributeHTML.java
@@ -68,18 +68,18 @@ final class AttributeHTML {
     }
 
 
-    final void close() {
+    void close() {
         file.println("</TABLE></BODY></HTML>");
         file.close();
     }
 
 
-    final void writeAttribute( final Attribute attribute, final String anchor ) {
+    void writeAttribute( final Attribute attribute, final String anchor ) {
         writeAttribute(attribute, anchor, 0);
     }
 
 
-    final void writeAttribute( final Attribute attribute, final String anchor, final int method_number ) {
+    void writeAttribute( final Attribute attribute, final String anchor, final int method_number ) {
         final byte tag = attribute.getTag();
         int index;
         if (tag == Const.ATTR_UNKNOWN) {
diff --git a/src/main/java/org/apache/bcel/util/ByteSequence.java b/src/main/java/org/apache/bcel/util/ByteSequence.java
index 73d2fdb..0a06577 100644
--- a/src/main/java/org/apache/bcel/util/ByteSequence.java
+++ b/src/main/java/org/apache/bcel/util/ByteSequence.java
@@ -37,12 +37,12 @@ public final class ByteSequence extends DataInputStream {
     }
 
 
-    public final int getIndex() {
+    public int getIndex() {
         return byteStream.getPosition();
     }
 
 
-    final void unreadByte() {
+    void unreadByte() {
         byteStream.unreadByte();
     }
 
@@ -52,12 +52,12 @@ public final class ByteSequence extends DataInputStream {
             super(bytes);
         }
 
-        final int getPosition() {
+        int getPosition() {
             // pos is protected in ByteArrayInputStream
             return pos;
         }
 
-        final void unreadByte() {
+        void unreadByte() {
             if (pos > 0) {
                 pos--;
             }


[commons-bcel] 05/06: Remove redundant semicolon.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit 745d93ca5551d7db9e87c5270473581144a91359
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:17:02 2019 -0400

    Remove redundant semicolon.
---
 src/test/java/org/apache/bcel/PerformanceTest.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/test/java/org/apache/bcel/PerformanceTest.java b/src/test/java/org/apache/bcel/PerformanceTest.java
index 404b699..66fa8b2 100644
--- a/src/test/java/org/apache/bcel/PerformanceTest.java
+++ b/src/test/java/org/apache/bcel/PerformanceTest.java
@@ -39,7 +39,7 @@ import junit.framework.TestCase;
 
 public final class PerformanceTest extends TestCase {
 
-    private static final boolean REPORT = Boolean.parseBoolean(System.getProperty("PerformanceTest.report", "true"));;
+    private static final boolean REPORT = Boolean.parseBoolean(System.getProperty("PerformanceTest.report", "true"));
 
     private static byte[] read(final InputStream is) throws IOException {
         if (is == null) {


[commons-bcel] 06/06: Oops, parent should be version 48, not a snapshot.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit 8cabe905c650f99cbffb6c0ed77dc0c34ca4d4f0
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:18:33 2019 -0400

    Oops, parent should be version 48, not a snapshot.
---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 21446e1..76f468f 100644
--- a/pom.xml
+++ b/pom.xml
@@ -27,7 +27,7 @@
   <parent>
     <groupId>org.apache.commons</groupId>
     <artifactId>commons-parent</artifactId>
-    <version>49-SNAPSHOT</version>
+    <version>48</version>
   </parent>
 
   <groupId>org.apache.bcel</groupId>


[commons-bcel] 03/06: Fix Javadocs.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit 53f2a5421cc49b3a085cc58be2cc638e141365b5
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:14:20 2019 -0400

    Fix Javadocs.
---
 src/main/java/org/apache/bcel/classfile/NestHost.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/NestHost.java b/src/main/java/org/apache/bcel/classfile/NestHost.java
index 68fdfcb..b261148 100644
--- a/src/main/java/org/apache/bcel/classfile/NestHost.java
+++ b/src/main/java/org/apache/bcel/classfile/NestHost.java
@@ -47,7 +47,7 @@ public final class NestHost extends Attribute {
     /**
      * @param name_index Index in constant pool
      * @param length Content length in bytes
-     * @param int Host class index
+     * @param host_class_index Host class index
      * @param constant_pool Array of constants
      */
     public NestHost(final int name_index, final int length, final int host_class_index,
@@ -88,7 +88,7 @@ public final class NestHost extends Attribute {
      * Dump NestHost attribute to file stream in binary format.
      *
      * @param file Output file stream
-     * @throws IOException
+     * @throws IOException if an I/O error occurs.
      */
     @Override
     public void dump( final DataOutputStream file ) throws IOException {
@@ -106,7 +106,7 @@ public final class NestHost extends Attribute {
 
 
     /**
-     * @param int the host class index
+     * @param host_class_index the host class index
      */
     public void setHostClassIndex( final int host_class_index ) {
         this.host_class_index = host_class_index;


[commons-bcel] 02/06: Fix spelling in Javadoc and such.

Posted by gg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git

commit 61b8686df10644780be6e4edc4d11fd73fb95f52
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Sep 1 18:13:24 2019 -0400

    Fix spelling in Javadoc and such.
---
 src/main/java/org/apache/bcel/classfile/ModuleMainClass.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java b/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
index 2de3582..153ecde 100644
--- a/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
+++ b/src/main/java/org/apache/bcel/classfile/ModuleMainClass.java
@@ -46,7 +46,7 @@ public final class ModuleMainClass extends Attribute {
     /**
      * @param name_index Index in constant pool
      * @param length Content length in bytes
-     * @param int Host class index
+     * @param main_class_index Host class index
      * @param constant_pool Array of constants
      */
     public ModuleMainClass(final int name_index, final int length, final int main_class_index,
@@ -71,7 +71,7 @@ public final class ModuleMainClass extends Attribute {
 
 
     /**
-     * Called by objects that are traversing the nodes of the tree implicitely
+     * Called by objects that are traversing the nodes of the tree implicitly
      * defined by the contents of a Java class. I.e., the hierarchy of methods,
      * fields, attributes, etc. spawns a tree of objects.
      *
@@ -87,7 +87,7 @@ public final class ModuleMainClass extends Attribute {
      * Dump ModuleMainClass attribute to file stream in binary format.
      *
      * @param file Output file stream
-     * @throws IOException
+     * @throws IOException if an I/O error occurs.
      */
     @Override
     public void dump( final DataOutputStream file ) throws IOException {
@@ -105,7 +105,7 @@ public final class ModuleMainClass extends Attribute {
 
 
     /**
-     * @param int the host class index
+     * @param main_class_index the host class index
      */
     public void setHostClassIndex( final int main_class_index ) {
         this.main_class_index = main_class_index;