jcheck review request - blacklist updates
Weijun Wang
weijun.wang at oracle.com
Wed Nov 28 18:21:27 PST 2012
On 11/29/2012 10:06 AM, John Coomes wrote:
> Weijun Wang (weijun.wang at oracle.com) wrote:
>> I'm not quite comfortable with:
>>
>> - '9eb97a2b3274edff83a362f76bbadb866a97a89b'
>> + '9eb97a2b3274edff83a362f76bbadb866a97a89b',
>
> Thanks for looking. The above is only the first bit of the diff hunk;
> the whole thing is:
>
> - '9eb97a2b3274edff83a362f76bbadb866a97a89b'
> + '9eb97a2b3274edff83a362f76bbadb866a97a89b',
> + # jdk7u11-dev/jaxp bad fix for 7192390
> + '1642814c94fd0206f8b4f460cc77fa6fc099731a',
> + # jdk7u11-dev/jdk bad fix for 7192390
> + '90eb0407ca69dc572d20490f17239b183bb616b6',
> + # jdk7u11-dev/jdk/test/closed bad fix for 7192390
> + '512af24c6909ef2c766c3a5217c719545de68bf7',
> + # jdk7u11-dev/jdk redone rmi fix
> + 'fd6ce0200a7f519380e6863930e92f9182030fa0',
> + # jdk7u11-dev/jdk/test/closed redone rmi fix
> + '770d9cf0e1dc97f7aaa3fdfbb430b27a40b1a3a9'
> ]
>
> There's no trailing comma.
>
>> In Java, a comma after the last array element is allowable. Hopefully
>> you can also do this in python.
>
> python doesn't allow it.
Maybe my python is too new?
$ python --version
Python 2.7.3
$ python -c 'print [1,2,3]'
[1, 2, 3]
$ python -c 'print [1,2,3,]'
[1, 2, 3]
If we were able to add a trailing comma, then any future diff would only
include the real new lines.
-Max
>
> -John
>
>> On 11/29/2012 07:54 AM, John Coomes wrote:
>>> Please review a simple change to add some cset hashes to the
>>> blacklist.
>>>
>>> http://cr.openjdk.java.net/~jcoomes/tools/jcheck/blacklist-20121128/
>>>
>>> -John
>>>
More information about the hg-tools-dev
mailing list