No superc #4
No superc #4
Conversation
| // TODO check generic type parameters | ||
|
|
||
| Field[] fields = clazz.getDeclaredFields(); | ||
| Field[] fields = clazz.getFields(); |
There was a problem hiding this comment.
What about protected fields? Don't we care about those?
There was a problem hiding this comment.
Damn it, I thought this method would return those. It does not. Not seeing an easy way to get inherited protected methods. Would have to recurse up. When recursing up, would need to dedupe taking 1st (handle case where subclass overrides superclass protected method and changes return type).
There was a problem hiding this comment.
Added partial support for protected methods. Thinking we can open an issue to cover protected methods in super classes. Really need test for tricky cases like this. I think it would be better to refactor the code to make it easier to test and then add support for protected in superclass with test.
No description provided.