eric-milles opened a new pull request #1661: URL: https://github.com/apache/groovy/pull/1661
This is not complete, but does give some shape to recent discussions. - `ASM_API_VERSION` goes to `AsmClassGenerator` - `DEFAULT_TARGET_BYTECODE` and `defaultTargetBytecode()` were unnecessary additions - `setTargetBytecode(String)` can use nearest value as described in GROOVY-10278 - `isPostJDKn` can be replaced with use of `isAtLeast(String,String)` -- the recent additions could be dropped, but I just moved them for now - `LoggableClassVisitor` --> `TraceClassVisitor` (just need to work out the jarjar stuff for unit tests Thoughts? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
