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.

Reply via email to