<i18n dev>  RFR: 8038092: Re-examine Bidi reflective dependency on java.awt.font
naoto.sato at oracle.com
Tue Jul 1 20:09:24 UTC 2014
Thanks, Alan. Here is the revised webrev based on your suggestions:
On 7/1/14, 6:58 AM, Alan Bateman wrote:
> On 30/06/2014 18:35, Naoto Sato wrote:
>> Please review the fix for the subject bug:
>> The proposed change is located at:
> Thanks for looking at this issue.
> One part that doesn't look right is where Bidi is used before
> TextAttribute or NumericShaper are initialized and then used later with
> one of these as an attribute. Normally with SharedSecrets then
> ensureClassInitialized is to used to initialize a class that is known to
> register the secret but in this case then you can't do that because it
> would create a dependency on java.awt. The simplest thing might be to
> keep the Class.forName in both TextAttribtueConstants and
> NumericShapings as that will ensure that those classes are initialized
> (if they are present).
> A minor comment is that there are probably a bunch of imports that can
> be removed once the bulk of the core reflection usage goes away.
More information about the i18n-dev