Patch: PR 16923

Thomas Fitzsimmons fitzsim@redhat.com
Wed Feb 23 22:46:00 GMT 2005


Hi,

On Wed, 2005-02-23 at 06:19 -0500, Bryce McKinlay wrote:
> OK, this looks good.
> 

I realized that the gij.cc change was no longer required since I
reallocate _Jv_Compiler_Properties in parse_init_args now.  I committed
this patch; the only differences are the removal of the gij.cc changes
and a change in the first comment in parse_init_args.  make check shows
no regressions.

Tom

2005-02-23  Thomas Fitzsimmons  <fitzsim@redhat.com>

	PR libgcj/16923
	* gcj.texi (Invocation): Add descriptions of JvVMInitArgs and
	JvVMOption.

2005-02-23  Thomas Fitzsimmons  <fitzsim@redhat.com>

	PR libgcj/16923
	* jni.cc (JNI_CreateJavaVM): Check JNI version.  Cast args to
	JvVMInitArgs.  Pass args to _Jv_CreateJavaVM and check return
	value.  Move argument parsing code to prims.cc.
	* prims.cc (no_properties): Remove.
	(_Jv_Compiler_Properties): Initialize to NULL.
	(_Jv_Properties_Count): Initialize to 0.
	(parse_verbose_args): New function.
	(parse_init_args): New function.
	(_Jv_CreateJavaVM): Call parse_init_args.
	(_Jv_RunMain): Check return value of _Jv_CreateJavaVM.
	* gcj/cni.h (JvVMOption): New struct.
	(JvVMInitArgs): Likewise.
	(JvCreateJavaVM): Declare vm_args as JvVMInitArgs* rather than
	void*.
	* libjava/gcj/javaprims.h (_Jv_VMOption): New struct.
	(_Jv_VMInitArgs): Likewise.
	* include/java-props.h (_Jv_Properties_Count): Declare.
	* java/lang/natRuntime.cc (insertSystemProperties): Use
	_Jv_Properties_Count in for loop exit condition.
	* testsuite/libjava.jni/jni.exp
	(gcj_invocation_compile_c_to_binary): New procedure.
	(gcj_invocation_test_one): Likewise.
	(gcj_jni_run): Run JNI invocation API tests.
	* testsuite/libjava.jni/invocation/PR16923.c,
	testsuite/libjava.jni/invocation/PR16923.java,
	testsuite/libjava.jni/invocation/PR16923.out: New test.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: libgcj-invocation6.patch
Type: text/x-patch
Size: 18377 bytes
Desc: 
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20050223/9e1625d9/attachment.bin>


More information about the Java-patches mailing list