On Wed, 20 Jan 2021 15:34:21 GMT, Claes Redestad <redes...@openjdk.org> wrote:
>> eirbjo has updated the pull request incrementally with one additional commit >> since the last revision: >> >> As a part of the rename from "canonize" to "canonicalize", doCanonize >> should also be renamed. This was an oversight in the original PR. > > LGTM - just a small nit that you seem to have forgotten to change canonize to > canonicalize in some places. Newbie question: Is a copyright year updates required when code is simply moved around like this? More specifically, is it required when strictly deleting code from a file, like ParseUtil in this case? ------------- PR: https://git.openjdk.java.net/jdk/pull/2167
