https://issues.apache.org/ooo/show_bug.cgi?id=124086
--- Comment #4 from h...@apache.org <h...@apache.org> --- Looking at the patch again there is a forward declaration of callVirtualMethod() immediately followed by its actual definition. IMHO the forward declaration is superfluous. I don't have a Solaris dev environment here, so I can't check, but from a C++ perspective only the "noinline" attribute needs to be moved and the forward declaration should be dropped. -- You are receiving this mail because: You are on the CC list for the bug. You are watching all bug changes.