Sure, I like the new simplified result.

Changes look fine,

Valerie


On 9/26/2018 7:09 PM, Weijun Wang wrote:
Webrev updated at https://cr.openjdk.java.net/~weijun/8210821/webrev.01.

On Sep 26, 2018, at 11:57 PM, Weijun Wang <weijun.w...@oracle.com> wrote:

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


Reply via email to