<div dir="ltr">Thanks Joel, that's great.<div><br></div><div>Liam</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 21, 2014 at 2:50 AM, Joel Borggren-Franck <span dir="ltr"><<a href="mailto:joel.franck@oracle.com" target="_blank">joel.franck@oracle.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Liam,<br>
<br>
Thanks!<br>
<br>
I filed <a href="https://bugs.openjdk.java.net/browse/JDK-8038080" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8038080</a><br>
<br>
I intend to submit your fix for 9 and 8u20.<br>
<br>
cheers<br>
<span class="HOEnZb"><font color="#888888">/Joel<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 2014-03-05, Liam Miller-Cushon wrote:<br>
> There appears to be a javac8 bug that prevents annotation processors from<br>
> visiting annotations on parameters. When repeated annotations were<br>
> implemented [1][2] the annotation scanners in JavacProcessingEnvironment<br>
> and JavacRoundEnvironment were modified and the following override was<br>
> added:<br>
><br>
> @Override<br>
> public Set<TypeElement> visitExecutable(ExecutableElement e,<br>
> Set<TypeElement> p) {<br>
>   // Type parameters are not considered to be enclosed by an executable<br>
>   scan(e.getTypeParameters(), p);<br>
>   return scan(e.getEnclosedElements(), p);<br>
> }<br>
><br>
> MethodSymbols don't have any enclosed elements, but they do have parameters<br>
> (which currently aren't being visited). I think that line was meant to<br>
> match the base implementation in ElementScanner:<br>
><br>
> return scan(e.getParameters(), p);<br>
><br>
><br>
> [1] <a href="https://bugs.openjdk.java.net/browse/JDK-7162089" target="_blank">https://bugs.openjdk.java.net/browse/JDK-7162089</a><br>
> [2] <a href="http://hg.openjdk.java.net/jdk8/jdk8/langtools/rev/1908e86ee49a" target="_blank">http://hg.openjdk.java.net/jdk8/jdk8/langtools/rev/1908e86ee49a</a><br>
</div></div></blockquote></div><br></div>