This is the mail archive of the java-patches@sources.redhat.com 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]

Re: [PATCH] New C++ ABI compatibility changes.


>>>>> "Alex" == Alexandre Petit-Bianco <apbianco@cygnus.com> writes:

Alex> 	(isAssignableFrom): Turned outline.
Alex> 	(isInstance): Likewise.
Alex> 	(isInterface): Likewise, fixed indentation.
Alex> 	(initializePrim): New function.

I forgot why these are needed.

Alex> +static java::lang::Class
Alex> +init_prim_class (jobject cname, jbyte sig, jint len, jobject array_vtable)
Alex> +{
Alex> +  static java::lang::Class iclass;
Alex> +  iclass.initializePrim (cname, sig, len, array_vtable);
Alex> +  return iclass;
Alex> +}

Now it occurs to me that maybe we could just have an initializing
constructor in Class instead of a no-arg constructor and a copy
constructor.  Then we wouldn't need this extra static class around.
Or, failing that, I also think we could just remove the `static' from
the iclass definition.

Tom

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