use MethodHandles instead of reflect.Method #4329
Conversation
|
can these be static final fields? those have special optimizations in hotspot |
they can't be static final the way the code is written at the moment but a bigger rewrite might be able to achieve that it looks like we won't be able to merge this anyway because animal-sniffer (Android compatibility checking) is rejecting the use of MethodHandles |
| @@ -91,10 +104,10 @@ public String[] getRecordFieldNames(Class<?> recordType) throws IllegalArgumentE | |||
| for (int i = 0; i < components.length; i++) { | |||
| try { | |||
| names[i] = (String) RECORD_COMPONENT_GET_NAME.invoke(components[i]); | |||
| } catch (Exception e) { | |||
| } catch (Throwable t) { | |||
There was a problem hiding this comment.
What would be the reason to catch Errors too?
There was a problem hiding this comment.
it's a required change unfortunately because invoke throws Throwable. this method shouldnt throw at all though
There was a problem hiding this comment.
Ok. Odd, not sure why it does that... never understood why method signature would do that (given Errors need not be declared anyway so why not just throws Exception). Oh well. At least this is for a reason then.
jackson-databind is very dependent on java reflection but we can start rewriting some pieces of code to avoid it