Re: [patch libjava]: Do some fixes for windows native targets

[Kai Tietz <ktietz70 at googlemail dot com>]

2011/4/6 Kai Tietz <ktietz70@googlemail.com>:
> 2011/4/5 Tom Tromey <tromey@redhat.com>:
>>>>>>> "Kai" == Kai Tietz <ktietz70@googlemail.com> writes:
>>
>> Kai> this patch fixes some issues in libjava for windows targets. ?It
>> Kai> addresses for now only 32-bit target, but for JNICALL definition I
>> Kai> introduced already logic for 64-bit windows target too as preparation.
>>
>> Most of this seems ok to me.
>>
>> Kai> ? ?* java/lang/reflect/Field.h (_Jv_JNI_ToReflectedField):
>> Kai> ? ?Likewise.
>> Kai> ? ?* java/lang/reflect/Constructor.h (_Jv_JNI_ToReflectedMethod):
>> Kai> ? ?Likewise.
>>
>> These headers are generated. ?You have to edit headers.txt and then
>> rebuild them.
>>
>> Tom
>>
>
> Hi Tom,
>
> I added to headers.txt the changes and regenerate headers. ?Btw it is
> a bit tricky to enable for libjava the maintainer-mode and then have a
> successful configure for it ...
>
> ChangeLog
>
> 2011-04-06 ?Kai Tietz
>
> ? ? ? ?* configure.ac (libgcj_ld_export_all): Set for windows
> ? ? ? ?native targets to export-all-symbols.
> ? ? ? ?(LIBGCJ_LD_EXPORT_ALL): Export libgcj_ld_export_all.
> ? ? ? ?* Makefile.am (libgcj_la_LDFLAGS): Add LIBGCJ_LD_EXPORT_ALL.
> ? ? ? ?* sysdep/i386/backtrace.h (MAIN_FUNC): Define for windows
> ? ? ? ?native targets to DllMain.
> ? ? ? ?* gcj/javaprims.h (JNICALL): Define it dependent
> ? ? ? ?for different windows targets.
> ? ? ? ?* include/jni_md.h: Likewise.
> ? ? ? ?* headers.txt (_Jv_JNI_ToReflectedField): Mark
> ? ? ? ?JNICALLS for friends.
> ? ? ? ?(_Jv_JNI_ToReflectedMethod): Likewise.
> ? ? ? ?* java/lang/Class.h: Regenerated.
> ? ? ? ?* java/lang/reflect/Field.h: Regenerated.
> ? ? ? ?* java/lang/reflect/Constructor.h: Regenerated.
> ? ? ? ?* configure: Regenerated.
> ? ? ? ?* Makefile.in: Regenerated.
> ? ? ? ?* include/Makefile.in: Regenerated.
> ? ? ? ?* testsuite/Makefile.in: Regenerated.
> ? ? ? ?* gcj/Makefile.in: Regenerated.
>
> Ok for apply?
>
> Regards,
> Kai
>

Sorry, found a typo. Updated patch attached.

Kai

Attachment: java_w32.txt
Description: Text document