Hi Valerie I've updated the CSR to remove the "always canonicalize" choice and now the default is "true", i.e. the current "smart-canonicalization" behavior. It's 12am here and I'll update the webrev tomorrow.
Thanks Max > On Sep 18, 2018, at 8:52 AM, Valerie Peng <valerie.p...@oracle.com> wrote: > > Look fine to me. Added myself to CSR as reviewer. > > Thanks, > > Valerie > > > On 9/17/2018 1:17 AM, Weijun Wang wrote: >> Please review the code change and CSR at >> >> http://cr.openjdk.java.net/~weijun/8210821/webrev.00/ >> https://bugs.openjdk.java.net/browse/JDK-8210822 >> >> Thanks >> Max >> >