RFR 8194268: Incorrect parameter names for synthetic methods

Jan Lahoda jan.lahoda at oracle.com
Tue Jan 30 14:38:07 UTC 2018


I've just pushed the change:
http://hg.openjdk.java.net/jdk/jdk/rev/b2f2bdba0472

Thanks for the patch and for the offer for help.

Jan

On 30.1.2018 14:42, Vicente Romero wrote:
> looks good, I can help pushing it
>
> Vicente
>
> On 01/26/2018 02:30 PM, Liam Miller-Cushon wrote:
>> Thanks! The changeset is attached.
>>
>> On Fri, Jan 26, 2018 at 10:34 AM, Jan Lahoda <jan.lahoda at oracle.com
>> <mailto:jan.lahoda at oracle.com>> wrote:
>>
>>     Sorry for delay. Seems OK to me, I'll run tests and push early
>>     next week.
>>
>>     Thanks,
>>          Jan
>>
>>     On 25.1.2018 19:27, Liam Miller-Cushon wrote:
>>
>>         Bump.
>>
>>         On Fri, Dec 29, 2017 at 2:24 PM, Liam Miller-Cushon
>>         <cushon at google.com <mailto:cushon at google.com>
>>         <mailto:cushon at google.com <mailto:cushon at google.com>>> wrote:
>>
>>             Hello,
>>
>>             This fixes a mistake in
>>         http://hg.openjdk.java.net/jdk/jdk/rev/89f6aa26fd6c
>>         <http://hg.openjdk.java.net/jdk/jdk/rev/89f6aa26fd6c>
>>             <http://hg.openjdk.java.net/jdk/jdk/rev/89f6aa26fd6c
>>         <http://hg.openjdk.java.net/jdk/jdk/rev/89f6aa26fd6c>>
>>             that caused incorrect parameter names to be generated for
>>         synthetic
>>             methods.
>>             As I mentioned in the bug I don't think it's a regression,
>>         but it
>>             still seems
>>             worth cleaning up.
>>
>>             bug: https://bugs.openjdk.java.net/browse/JDK-8194268
>>         <https://bugs.openjdk.java.net/browse/JDK-8194268>
>>             <https://bugs.openjdk.java.net/browse/JDK-8194268
>>         <https://bugs.openjdk.java.net/browse/JDK-8194268>>
>>             webrev:
>>         http://cr.openjdk.java.net/~cushon/8194268/webrev.00/
>>         <http://cr.openjdk.java.net/%7Ecushon/8194268/webrev.00/>
>>             <http://cr.openjdk.java.net/~cushon/8194268/webrev.00/
>>         <http://cr.openjdk.java.net/%7Ecushon/8194268/webrev.00/>>
>>
>>             Thanks,
>>             Liam
>>
>>
>>
>


More information about the compiler-dev mailing list