RFR: JDK-8207160: ClassReader::adjustMethodParams can potentially return null if the args list is empty

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Wed Sep 12 20:41:29 UTC 2018


There's no copyright on the jasm file - is that deliberate?

Thanks
Maurizio


On 12/09/18 21:23, Liam Miller-Cushon wrote:
> Looks good, thanks!
>
> On Wed, Sep 12, 2018 at 12:36 PM Vicente Romero 
> <vicente.romero at oracle.com <mailto:vicente.romero at oracle.com>> wrote:
>
>     Hi Liam,
>
>     Thanks for your comments, I have updated the patch [1],
>
>     [1]
>     http://cr.openjdk.java.net/~vromero/8207160/webrev.01/jdk.dev.patch
>     <http://cr.openjdk.java.net/%7Evromero/8207160/webrev.00/jdk.dev.patch>
>
>     On 09/10/2018 06:54 PM, Liam Miller-Cushon wrote:
>>     Hi Vicente,
>>
>>     This looks good to me, for what that's worth.
>>
>>     A couple of nits:
>>     * Consider doing the `args.isEmpty()` check earlier, at the
>>     beginning of adjustMethodParams or even before it's called. It
>>     might be slightly clearer, and it avoids processing the empty
>>     list and creating an unnecessary ListBuffer.
>>     * I thought more descriptive test names were now preferred over
>>     T8207160-style names?
>>
>>     On Mon, Sep 10, 2018 at 2:13 PM Vicente Romero
>>     <vicente.romero at oracle.com <mailto:vicente.romero at oracle.com>> wrote:
>>
>>         ping
>>
>>         On 08/31/2018 03:18 PM, Vicente Romero wrote:
>>         > Please review the fix for [1] at [2]. The bug was found by
>>         a user that
>>         > was compiling some classes with JDK11 while having some old
>>         libs built
>>         > with JDK6 in the class path. It seems like some versions of
>>         javac6
>>         > produced constructors for inner classes with no arguments.
>>         This is not
>>         > expected by newer versions of javac and it was producing a
>>         NPE. This
>>         > fix aims at making sure that
>>         ClassReader::adjustMethodParams wont
>>         > return a null even if the list of arguments passed to it is
>>         empty.
>>         >
>>         > TIA,
>>         > Vicente
>>         >
>>         > [1] https://bugs.openjdk.java.net/browse/JDK-8207160
>>         > [2]
>>         http://cr.openjdk.java.net/~vromero/8207160/webrev.00/jdk.dev.patch
>>         <http://cr.openjdk.java.net/%7Evromero/8207160/webrev.00/jdk.dev.patch>
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20180912/34cc8656/attachment-0001.html>


More information about the compiler-dev mailing list