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 08:42:00 UTC 2018
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.
/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