<AWT Dev> [9] Review Request: JDK-8038106 PIT: access problem to a PrinterJob on OS X

[Alexander Zvegintsev]

Hi Petr,

The fix looks good to me as well.

--
Thanks,
Alexander.

21.03.2014 18:28, Petr Pchelko wrote:
> Hello, AWT Team.
>
> Please review the fix for the issue: 
> https://bugs.openjdk.java.net/browse/JDK-8038106
> The fix is available here: 
> http://cr.openjdk.java.net/~pchelko/9/8038106/webrev/ 
> <http://cr.openjdk.java.net/%7Epchelko/9/8038106/webrev/>
>
> Apparently I've made the class package-local and did not notice that 
> the class is being called by reflection.
> I've checked other classes I've restricted access too, they should be 
> fine. I've also verified that KSS
> would not complain if we mace the class public as it does not have 
> public native methods. The only
> protected native method is fine as the class is final.
>
> With best regards. Petr.
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20140321/ca43db43/attachment-0001.html>