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]
Other format: [Raw text]

Re: Patch for Preview: _Jv_ThisExecutable (The Saga Continues)


Hi Andrew,

>This breaks JvCreateJavaVM, because _Jv_argv isn't necessarily set.
>Please check _Jv_argv != NULL before doing this.

Sorry about this. I'll redo the patch and resubmit it ASAP. I'll need to retest
the various build combinations so I'll shoot for submitting this sometime tomorrow
at the latest.

>BTW, is there any reason 
>
> > +extern int _Jv_argc;
> > +extern const char **_Jv_argv;
>
>isn't in some header file?

You're right about this too. I'll do this in the patch I resubmit.

Sorry again for the inconvenience.

-- Mohan
http://www.thisiscool.com/
http://www.animalsong.org/





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