Hi Phil,

Please find the modified webrev with the testcase:
http://cr.openjdk.java.net/~psadhukhan/6197099/webrev.01/

Regards
Prasanta
On 4/13/2016 8:05 PM, Philip Race wrote:
This needs a test. You can provide one by following other similar examples.
Also it needs a CCC to document the exception.
Finally, I am a bit surprised at the exact exception.
Where precisely does this originate (stack trace, please) ?
Probably we should throw SecurityException instead, even
if it is not the exact behaviour today.
Just make sure that our own code is ready for such a change.

-phil.

On 4/12/16, 10:23 PM, prasanta sadhukhan wrote:
Resending with bugid in the subject line.

On 4/6/2016 5:06 PM, prasanta sadhukhan wrote:
Hi All,

Please review a document fix for jdk9
Bug: https://bugs.openjdk.java.net/browse/JDK-6197099
webrev: http://cr.openjdk.java.net/~psadhukhan/6197099/webrev.00/

PrinterJob.getUserName() is supposed to return the current user name of the system. This method does not document any exception. But when SecurityManager is installed, this method throws a security exception when PropertyPermission - user.name is not given in the policy file. But this is not documented anywhere and hence the user will not know until he gets the exception.

Documented the long-standing existing exception.

will raise a ccc after the technical review.

Regards
Prasanta


Reply via email to