RFR 9: 8048124 Read hijrah-config-umalqura.properties as a resource

Alan Bateman Alan.Bateman at oracle.com
Mon Oct 20 15:07:56 UTC 2014


On 20/10/2014 15:53, roger riggs wrote:
> Please review for JDK 9 only.
>
> To aid the modularization effort, the configuration of the Hijrah 
> calendar
> should move the Hijrah calendar data to a resource.
>
> At this point, it does not look like there will be other Hijrah calendar
> variants; the function of calendar.properties to configure variants
> is unnecessary and is proposed to be removed.
>
> Since the other uses of calendars.properties have been eliminated
> the calendars.properties is removed.
>
> Webrev:
>   http://cr.openjdk.java.net/~rriggs/webrev-hijrah-config-8049376/
> Issue:
>   https://bugs.openjdk.java.net/browse/JDK-8048124
This mostly looks good to me. I just wonder about the removal of the 
doPrivileged in readConfigProperties where you will get null if there is 
something on the stack with restricted permissions.

A minor comment on the @implNote in HijrahChronology is that it has more 
than I would expect, it might be simpler to just say that 
hijrah-config-<calendar-type>.properties is loaded as a resource file. 
Also the import of java.lang.String looks unnecessary.

-Alan.





More information about the core-libs-dev mailing list