[Patch] java.lang.System - comments fixed

Michael Koch konqueror@gmx.de
Tue Apr 26 05:14:00 GMT 2005


Hi list,


I just commited the attached patch to merge one comment from GNU
classpath and to remove another. This gets java.lang.System more in line
with GNU classpath.


Michael


2005-04-26  Michael Koch  <konqueror@gmx.de>

	* java/lang/System.java
	(setSecurityManager): Fixed comment.
	(getSecurityManager): Removed obsolete comment.

-------------- next part --------------
Index: java/lang/System.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/java/lang/System.java,v
retrieving revision 1.24
diff -u -r1.24 System.java
--- java/lang/System.java	25 Apr 2005 20:35:17 -0000	1.24
+++ java/lang/System.java	26 Apr 2005 05:09:04 -0000
@@ -177,9 +177,9 @@
    */
   public static synchronized void setSecurityManager(SecurityManager sm)
   {
-    // Implementation note: the field lives in Runtime because of bootstrap
-    // initialization issues. This method is synchronized so that no other
-    // thread changes it to null before this thread makes the change.
+    // Implementation note: the field lives in SecurityManager because of
+    // bootstrap initialization issues. This method is synchronized so that
+    // no other thread changes it to null before this thread makes the change.
     if (SecurityManager.current != null)
       SecurityManager.current.checkPermission
         (new RuntimePermission("setSecurityManager"));
@@ -194,8 +194,6 @@
    */
   public static SecurityManager getSecurityManager()
   {
-    // Implementation note: the field lives in Runtime because of bootstrap
-    // initialization issues.
     return SecurityManager.current;
   }
 


More information about the Java-patches mailing list