This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Patch: FYI: re-merge AccessibleObject


I'm checking this in.  It updates the AccessibleObject copyright info,
and adds some missing code and a couple FIXME comments.

Tom

Index: ChangeLog
from  Tom Tromey  <tromey@redhat.com>

	* java/lang/reflect/AccessibleObject.java (checkPermission):
	Implemented.
	Updated copyright information.

Index: java/lang/reflect/AccessibleObject.java
===================================================================
RCS file: /cvs/gcc/gcc/libjava/java/lang/reflect/AccessibleObject.java,v
retrieving revision 1.4
diff -u -r1.4 AccessibleObject.java
--- java/lang/reflect/AccessibleObject.java 2000/11/18 02:30:29 1.4
+++ java/lang/reflect/AccessibleObject.java 2001/10/15 22:43:59
@@ -1,12 +1,29 @@
-// AccessibleObject.java - Base for reflection objects.
+/* java.lang.reflect.AccessibleObject
+   Copyright (C) 1998, 1999, 2000, 2001 Free Software Foundation, Inc.
 
-/* Copyright (C) 1998, 1999, 2000  Free Software Foundation
+This file is part of GNU Classpath.
 
-   This file is part of libgcj.
+GNU Classpath is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+ 
+GNU Classpath is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with GNU Classpath; see the file COPYING.  If not, write to the
+Free Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
+02111-1307 USA.
+
+As a special exception, if you link this library with other files to
+produce an executable, this library does not by itself cause the
+resulting executable to be covered by the GNU General Public License.
+This exception does not however invalidate any other reasons why the
+executable file might be covered by the GNU General Public License. */
 
-This software is copyrighted work licensed under the terms of the
-Libgcj License.  Please consult the file "LIBGCJ_LICENSE" for
-details.  */
  
 package java.lang.reflect;
 
@@ -33,6 +50,9 @@
   public static void setAccessible (AccessibleObject[] array, boolean flag)
   {
     checkPermission ();
+    // FIXME: check for invalid changes in the loop.
+    // For instance, can't set this flag to true for a Constructor for
+    // Class (example from the manual).
     for (int i = 0; i < array.length; ++i)
       array[i].flag = flag;
   }
@@ -40,13 +60,17 @@
   public void setAccessible (boolean flag)
   {
     checkPermission ();
+    // FIXME: check for invalid changes.
+    // For instance, can't set this flag to true for a Constructor for
+    // Class (example from the manual).
     this.flag = flag;
   }
 
   private static final void checkPermission ()
   {
     SecurityManager sm = System.getSecurityManager();
-    // FIXME: sm.checkPermission(ReflectPermission ("suppressAccessChecks"))
+    if (sm != null)
+      sm.checkPermission (new ReflectPermission ("suppressAccessChecks"));
   }
 
   private boolean flag;


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]