Tom Tromey wrote: > I'm posting this for comment. > > This is the patch to change Method/Field/Constructor to have a > 'getModifiersInternal' method, which returns the un-masked modifiers > as read from the .class file. This lets us implement the new > 1.5 reflection predicates such as isSynthetic.
Looks good to me. > Note that we could also do something similar to this for Class. I > think we ought to but I'd prefer to do it as a separate patch. VMClass.getModifiers() already should return the raw modifiers. Of course, for Class there is the added complication of inner classes that have an additional modifiers word (which ATM is selected by a boolean passed to VMClass.getModifiers, which is a bit ugly). Regards, Jeroen
