Details
Description
Re.
http://qdox.codehaus.org/apidocs/com/thoughtworks/qdox/model/JavaClass.html#getInnerClasses()
The method getInnerClasses() seems to actually be listing all nested classes, including static nested classes, not just inner classes. This is a common misuse of the term "inner class". For definitions see
http://java.sun.com/docs/books/jls/second_edition/html/classes.doc.html
and search for "nested" and "inner", specifically
"An inner class is a nested class that is not explicitly or implicitly declared static."
You might consider renaming the method to JavaClass.getNestedClasses() for this reason, unless you are really returning only inner classes.
Activity
| Field | Original Value | New Value | 
|---|---|---|
| Fix Version/s | 1.6 [ 10814 ] | 
| Assignee | Mike Williams [ mdub ] | 
| Resolution | Fixed [ 1 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] | 
                
Good point, thanks. We should rename it (and retain getInnerClasses as a deprecated synonym).