RFR 8117883: nasgen prototype, instance member count calculation is wrong

Hannes Wallnoefer hannes.wallnoefer at oracle.com
Thu Jun 18 13:18:30 UTC 2015


Looks good.


Am 2015-06-18 um 14:57 schrieb A. Sundararajan:
> Hi,
>
> Please review updated review 
> http://cr.openjdk.java.net/~sundar/8117883/webrev.01/
>
> PS. Fixed for what Ahmed found in his previous email as well as 
> cleanup other things (like not adding CONSTRUCTOR, 
> SPECIALIZED_FUNCTION (constructor) as members for Constructor class) etc.
>
> Thanks,
> -Sundar
>
> On Thursday 18 June 2015 11:57 AM, Ahmed Ashour wrote:
>> Hi Sundar,
>> PropertyGenerator is ok, but ConstructorGenerator still has the issue.
>> For example:
>> NativeArray:
>>     static {      ArrayList list = new ArrayList(7); 
>> list.add(AccessorProperty.create("isArray", 2, "G$isArray", 
>> "S$isArray"));      $nasgenmap$ = PropertyMap.newMap(list);   }
>> because ScriptClassInfo.getConstructorMemberCount() returns also 
>> Kind.CONSTRUCTOR and .SPECIALIZED_FUNCTION, which are not 'added'.
>> Thanks a lot,Ahmed      From: Ahmed Ashour <asashour at yahoo.com>
>>   To: A. Sundararajan <sundararajan.athijegannathan at oracle.com>; 
>> "nashorn-dev at openjdk.java.net" <nashorn-dev at openjdk.java.net>
>>   Sent: Wednesday, June 17, 2015 9:41 PM
>>   Subject: Re: RFR 8117883: nasgen prototype, instance member count 
>> calculation is wrong
>>     Hi Sundar,
>> Works great, thanks for the quick response.
>> I don't fully understand the removal of ScriptObject.isGenericDescriptor
>> I guess users are not allowed to be created in bugs.openjdk.java.net.
>> Thanks again,Ahmed
>>
>>       From: A. Sundararajan <sundararajan.athijegannathan at oracle.com>
>>   To: "nashorn-dev at openjdk.java.net" <nashorn-dev at openjdk.java.net>
>>   Sent: Wednesday, June 17, 2015 7:20 PM
>>   Subject: RFR 8117883: nasgen prototype, instance member count 
>> calculation is wrong
>>     Please review http://cr.openjdk.java.net/~sundar/8117883/ for
>> https://bugs.openjdk.java.net/browse/JDK-8117883
>>
>> Thanks,
>> -Sundar
>>
>>
>>
>



More information about the nashorn-dev mailing list