<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Jul 6, 2018 at 7:24 PM Werner Dietl <<a href="mailto:wdietl@gmail.com" target="_blank">wdietl@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Does this sound good to you?<br></blockquote><div><br></div><div>Yes, I agree with your summary. Thanks for the corrections and clarifications.</div><div><br></div><div><span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">In particular I agree that speaking of 'duplicating' annotations is incorrect, 'propagating' seems to be a better word. Parts of the annotation structures are copied when propagating annotations from the anonymous class supertype to the new expression (e.g. the element value pair structure), but not the target_type or type_path.</span><br></div><div> </div><div>I updated the fix to only propagate annotations from the top-level supertype. For example, given `new @A Outer. @B Inner<@C Object>() {}` we propagate only `@B`. When the annotations are propagated an appropriate location is generated with one or more INNER_TYPE entries to place the annotation on the anonymous' class synthetic declaration type.</div><div><br></div><div>Here's the latest webrev: <a href="http://cr.openjdk.java.net/~cushon/8198945/webrev.03/" target="_blank">http://cr.openjdk.java.net/~cushon/8198945/webrev.03/</a></div><div><br></div><div>While continuing to investigate this I think I discovered a couple of related bugs:</div><div><span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">* </span><a href="https://bugs.openjdk.java.net/browse/JDK-8207017" style="color:rgb(17,85,204);font-size:small;background-color:rgb(255,255,255)" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8207017</a><br></div><div>* <a href="https://bugs.openjdk.java.net/browse/JDK-8207018" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8207018</a></div><div><br></div><div>I can continue to iterate on the fix to try to fix those issues as well, but they might be distinct enough to be worth dealing with separately.</div></div></div>