Hi Kiran,
1926 int sn = s.length();
1927 int tn = t.length();
could be moved before line 1921 which could then become:
if (testForEquality && sn != tn) return sn - tn;
Otherwise looks good.
best regards,
-- daniel
On 13/12/2019 19:28, Kiran Ravikumar wrote:
Hi Guys,
Could someone please review my fix to URI.java class were
URI.compareTo(URI) behavior was different to URI.equals(URI).
|URI.compareTo|(URI) does not consider two URI's to be equal when they
differ only in the case of hexadecimal digits of escaped octets, while
|URI.equals(URI)|does consider such URIs to be equal.
The fix involves spec and code change. A CSR was filed and approved.
Please find the webrev at -
http://cr.openjdk.java.net/~kravikumar/5064980/webrev.0/
JBS: https://bugs.openjdk.java.net/browse/JDK-5064980
Corresponding test is also updated to verify the behavior.
Thanks,
Kiran
