Review Request JDK-8232617: Update the outdated code comments in java.lang.System class
Mandy Chung
mandy.chung at oracle.com
Sat Oct 19 00:38:11 UTC 2019
thanks. I think "separated from" reads right to me. I will leave it as is.
Mandy
On 10/18/19 2:46 PM, Brent Christian wrote:
> Looks fine. You might consider s/separated/separately/ .
>
> -Brent
>
> On 10/18/19 1:56 PM, Mandy Chung wrote:
>> A trivial doc fix:
>>
>>
>> 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
>> @@ -94,10 +94,8 @@
>> public final class System {
>> /* Register the natives via the static initializer.
>> *
>> - * VM will invoke the initializeSystemClass method to complete
>> - * the initialization for this class separated from clinit.
>> - * Note that to use properties set by the VM, see the constraints
>> - * described in the initializeSystemClass method.
>> + * VM will invoke the initPhase1 method to complete the
>> initialization
>> + * for this class separated from <clinit>.
>> */
>> private static native void registerNatives();
>> static {
>>
>>
>>
>> Thanks
>> Mandy
>>
More information about the core-libs-dev
mailing list