Looks good to me; +1

On Jul 16, 2013, at 6:03 PM, Konstantin Shefov <konstantin.she...@oracle.com> 
wrote:

> Please, review the two exception list solution:
> 
> http://cr.openjdk.java.net/~kshefov/8017585/webrev.01/ 
> <http://cr.openjdk.java.net/%7Ekshefov/8017585/webrev.01/>
> 
> 16.07.2013 16:13, Konstantin Shefov пишет:
>> I will try to implement this.
>> 
>> On 16-Jul-13 16:11, Jim Laskey (Oracle) wrote:
>>> We still expect a two exception list solution at some point.
>>> 
>>> On 2013-07-16, at 8:50 AM, A. Sundararajan 
>>> <sundararajan.athijegannat...@oracle.com> wrote:
>>> 
>>>> And please make sure "ant clean test" and "ant clean test262parallel" are 
>>>> fine with your change before pushing the change.
>>>> 
>>>> -Sundar
>>>> 
>>>> On Tuesday 16 July 2013 04:33 PM, Jim Laskey (Oracle) wrote:
>>>>> We had a discussion about whether this was the best approach. I think we 
>>>>> had been thinking in terms of two separate exclude lists. In lieu of a 
>>>>> better solution, go a head with this change.
>>>>> 
>>>>> Cheers,
>>>>> 
>>>>> -- Jim
>>>>> 
>>>>> 
>>>>> 
>>>>> On 2013-07-16, at 7:33 AM, Konstantin Shefov 
>>>>> <konstantin.she...@oracle.com> wrote:
>>>>> 
>>>>>> REMINDER
>>>>>> On 11-Jul-13 14:41, Konstantin Shefov wrote:
>>>>>>> Hello,
>>>>>>> 
>>>>>>> Please review a fix for the issue:
>>>>>>> 
>>>>>>> 8017585 Exclude two failing tests from Nashorn CC run
>>>>>>> 
>>>>>>> http://bugs.sun.com/view_bug.do?bug_id=8017585
>>>>>>> 
>>>>>>> The webrev is: http://cr.openjdk.java.net/~kshefov/8017585/webrev.00
>>>>>>> 
>>>>>>> Thanks,
>>>>>>> Konstantin
>> 
> 

Reply via email to