matrei commented on code in PR #2267:
URL: https://github.com/apache/groovy/pull/2267#discussion_r2217020381
##########
src/main/java/org/codehaus/groovy/classgen/AsmClassGenerator.java:
##########
@@ -2221,10 +2222,11 @@ private AnnotationVisitor getAnnotationVisitor(final
AnnotatedNode targetNode, f
* @param av the visitor to use
*/
public void visitAnnotationAttributes(final AnnotationNode an, final
AnnotationVisitor av) {
- Map<String, Object> constantAttrs = new HashMap<>();
- Map<String, PropertyExpression> enumAttrs = new HashMap<>();
- Map<String, Object> atAttrs = new HashMap<>();
- Map<String, ListExpression> arrayAttrs = new HashMap<>();
+ // GROOVY-11715 TODO If we can determine what was causing the issues
mentioned on that ticket, we should change these back to LinkedHashMap
Review Comment:
@paulk-asert Would a plausible explanation to why `LinkedHashMap` isn't
working be that the items are inserted into the maps in different order across
different runs? As `LinkedHashMap` maintains insertion order.
--
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: notifications-unsubscr...@groovy.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
