You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@commons.apache.org by bu...@apache.org on 2003/06/20 01:56:48 UTC

DO NOT REPLY [Bug 20931] New: - Addition of static class modifer to inner classes

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20931>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20931

Addition of static class modifer to inner classes

           Summary: Addition of static class modifer to inner classes
           Product: Commons
           Version: Nightly Builds
          Platform: All
               URL: http://www.cs.umd.edu/~pugh/java/bugs/
        OS/Version: All
            Status: NEW
          Severity: Minor
          Priority: Other
         Component: Collections
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: janekdb@yahoo.co.uk


findbugs 0.6.0 suggested that these classes had inner classes which could be
made static

    CursorableLinkedList.java
    DefaultMapBag.java
    ExtendedProperties.java
    StaticBucketMap.java

The attached patch file patches these classes in that way. After patching the
unit test passed okay.

I'm assuming the unit tests passing is proof enough that these changes are okay.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org