RFR for CODETOOLS-7902293 and 7902317 jcheck extension broken in recent builds of hg
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Thu Sep 20 18:51:09 UTC 2018
On 2018-09-20 16:14, Tim Bell wrote:
> Magnus:
>
>> On 2018-09-20 06:47, Tim Bell wrote:
>>> Hello
>>>
>>> Two changes in recent builds of hg broke jcheck:
>>>
>>> 1) CODETOOLS-7902292: util.Abort() was moved/copied from util to error
>>> in hg 1.3 and was removed from util in 4.6. This is the same fix used
>>> on trees.py back in July [1].
>>>
>>> 2) CODETOOLS-7902317: date-related utils moved from util to
>>> utils/dateutil in early 2018 (hg 4.7).
>>>
>>> http://cr.openjdk.java.net/~tbell/CODETOOLS-7902293-7902317/webrev/
>>
>> Basically looks good. (But I'm still not a formal Reviewer :-())
>>
>> However, I'm thinking that -- in line with the name error_Abort for
>> error.Abort -- maybe an even better name than datestr_F would be
>> dateutil_datestr? The "_F" part (for function?) looks a bit odd.
>
> Yes, it was for function. Thanks for the suggestion - I was
> struggling with the name and decided to go with what I had.
>
> I like your name better. New webrev:
>
> http://cr.openjdk.java.net/~tbell/CODETOOLS-7902293-7902317/01/webrev/
LGTM.
/Magnus
>
> Tim
>
>>
>> /Magnus
>>
>>>
>>> Tested with hg versions 0.9.5, 2.9, 3.2.3, 4.7
>>>
>>> Thanks in advance. I will also need a sponsor to push the change
>>> for me.
>>>
>>> Tim
>>>
>>> [1] Ref lines 140 ... 146:
>>> http://hg.openjdk.java.net/code-tools/trees/comparison/592eba45b774/trees.py
>>>
>>>
>>
>
More information about the code-tools-dev
mailing list