natField.cc (getAddr): Allow setting of final field when it is accessible.

	* java/lang/reflect/natField.cc (getAddr): Allow setting of final
	field when it is accessible.

From-SVN: r114896

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index c1dcf661140fd5784edfafc2a50fda0d75374d62..071d27df61afeb7537c225d672d353272626ef0f 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2006-06-22  Tom Tromey  <tromey@redhat.com>
+
+       * java/lang/reflect/natField.cc (getAddr): Allow setting of final
+       field when it is accessible.
+
 2006-06-21  Bryce McKinlay  <mckinlay@redhat.com>
 
        * include/win32-threads.h (_Jv_ThreadDesc_t): New typedef. 

diff --git a/libjava/java/lang/reflect/natField.cc b/libjava/java/lang/reflect/natField.cc
index ae68639065b58f20cb46e7aba31f59b14bd4f758..487f62ab0e2a1f64aed030b3d37ad402ae5bcd9a 100644 (file)
--- a/libjava/java/lang/reflect/natField.cc
+++ b/libjava/java/lang/reflect/natField.cc
@@ -72,6 +72,10 @@ getAddr (java::lang::reflect::Field* field, jclass caller, jobject obj,
 
   // Setting a final field is usually not allowed.
   if (checkFinal
+      // As of 1.5, you can set a non-static final field if it is
+      // accessible.
+      && (! field->isAccessible()
+         || (field->getModifiers() & java::lang::reflect::Modifier::STATIC))
       && (field->getModifiers() & java::lang::reflect::Modifier::FINAL))
     throw new java::lang::IllegalAccessException(JvNewStringUTF 
       ("Field is final"));