Hi All,
Please review updated webrev
http://cr.openjdk.java.net/~srastogi/8158510/webrev.03/
Regards,
Shilpi
On 7/1/2016 5:11 PM, shilpi.rast...@oracle.com wrote:
Thanks Paul !!
Please see http://cr.openjdk.java.net/~srastogi/8158510/webrev.01/
Regards,
Shilpi
On 7/1/2016 3:12 PM, Paul Sandoz wrote:
Hi Shilpi,
There is more going on here than just the test since you have
modified the annotation processing to throw an ISE for an annotation
type that contains one or more methods that do not define elements.
That behaviour might be too restrictive.
This is a grey area and implementation specific but i would tend to
be cautious and avoid changing the current behaviour. Note that the
original issue was due to javac adding a synthetic de-sugared method
for a lambda. We don’t know if other byte code generators might do
something different.
So my recommendation is to limit the testing to the current set of
possible failures.
Paul.
On 1 Jul 2016, at 10:15, shilpi.rast...@oracle.com
<mailto:shilpi.rast...@oracle.com> wrote:
Hi All,
Please review https://bugs.openjdk.java.net/browse/JDK-8158510
*Problem:* How to validate annotation, as javac does not allow us to
write wrong annotation so how should we test valid annotation at
runtime?
http://cr.openjdk.java.net/~srastogi/8158510/webrev.00/
*Solution:* To test this i used ASM tool and modified the classfile
with wrong annotation (not allowed at javac level) and wrote test
cases.
Thanks,
Shilpi