RFR 8216205: Java API documentation formatting error in System.getEnv
Brent Christian
brent.christian at oracle.com
Mon Jan 7 19:03:59 UTC 2019
Also, AFAICT all the other javadoc mentions of RuntimePermission
(besides "getenv") use @code, rather than @link.
-Brent
On 1/7/19 10:58 AM, Brent Christian wrote:
> Looks good, though it looks like a similar change is needed on L987 for
> System.getenv(name).
>
> -Brent
>
> On 1/7/19 10:45 AM, Roger Riggs wrote:
>> Please review a javadoc fix. Only @link is needed, @code is removed.
>>
>>
>> diff --git a/src/java.base/share/classes/java/lang/System.java
>> b/src/java.base/share/classes/java/lang/System.java
>> --- a/src/java.base/share/classes/java/lang/System.java
>> +++ b/src/java.base/share/classes/java/lang/System.java
>> @@ -1,5 +1,5 @@
>> /*
>> - * Copyright (c) 1994, 2018, Oracle and/or its affiliates. All rights
>> reserved.
>> + * Copyright (c) 1994, 2019, Oracle and/or its affiliates. All rights
>> reserved.
>> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>> *
>> * This code is free software; you can redistribute it and/or modify it
>> @@ -1055,7 +1055,7 @@ public final class System {
>> * <p>If a security manager exists, its
>> * {@link SecurityManager#checkPermission checkPermission}
>> * method is called with a
>> - * {@code {@link RuntimePermission}("getenv.*")} permission.
>> + * {@link RuntimePermission RuntimePermission("getenv.*")}
>> permission.
>> * This may result in a {@link SecurityException} being thrown.
>> *
>> * <p>When passing information to a Java subprocess,
>>
>>
>> Thanks, Roger
More information about the core-libs-dev
mailing list