+1

> On 03 Nov 2015, at 15:41, Attila Szegedi <attila.szeg...@oracle.com> wrote:
> 
> +1
> 
>> On Nov 3, 2015, at 3:32 PM, Sundararajan Athijegannathan 
>> <sundararajan.athijegannat...@oracle.com> wrote:
>> 
>> Also piggybacking to add a sample (find_underscores.js)
>> 
>> -Sundar
>> 
>> On 11/3/2015 8:01 PM, Sundararajan Athijegannathan wrote:
>>> Please review http://cr.openjdk.java.net/~sundar/8141285/webrev.00/ for 
>>> https://bugs.openjdk.java.net/browse/JDK-8141285
>>> 
>>> NameCodec has similar functionality (with a minor bug in '\' handling) as 
>>> in sun.invoke.util.BytecodeName.java (which seems to be old code, but 
>>> unused in jdk repo!). Using code in BytecodeName.java in NameCodec.java now 
>>> - only minimal subset needed for decode, encode. Also, adding tests for 
>>> NameCodec.java derived from (older) mlvm test BytecodeNameTest.java
>>> 
>>> Thanks,
>>> -Sundar
>> 
> 

Reply via email to