On 20/10/2014 20:11, roger riggs wrote:
Hi,
Updated with recommendations to make the code more readable and
to use the more appropriate Integer.parseInt instead of valueOf.
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-hijrah-config-8049376/
Issue:
https://bugs.openjdk.java.net/browse/JDK-8048124
This looks okay now although I guess the other clean-ups are really
something for a different issue.
You want to want to check Integer.java in the webrev, I assumed that
isn't meant to be there.
-Alan.