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.


Alexandre Petit-Bianco wrote:

> Tom Tromey writes:
>
> > I forgot why these are needed.
>
> Otherwise, they disapear from the symbol table and we get a link
> error. YMMV, on some systems, only isAssignableFrom needs to be
> altered. I need to investigate.

Well, I think basically its illegal C++ to declare something inline
and not define it in all translation units where it is used. So the
code was always wrong, its just that problems happen to show up under
the new ABI for some reason.

Thats okay though, because inlining these functions is not really very
important for performance, except for isInterface(), which can be
moved to Class.h.

regards

  [ bryce ]



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