-
Notifications
You must be signed in to change notification settings - Fork 30
Open
Description
I don't know Java well, so I'd just like to leave this as a suggestion, without judgement:
Our static analysis suggests a change at https://github.com/qmx/jitescript/blob/master/src/main/java/me/qmx/jitescript/VisibleAnnotation.java#L34, to change value.getClass() to value.getDeclaredClass(). The former returns some derived class for enums that declare methods, whereas the latter returns the actual enum type.
I'm not sure if that's correct, but it was flagged as an error-prone pattern.
Metadata
Metadata
Assignees
Labels
No labels