All right then. I'm fine with the fix.
--
best regards,
Anthony
On 4/12/2013 19:22, David DeHaven wrote:
I totally agree that the current usage of JNIEXPORT in typedefs in AWT and 2D
code is a bug.
My concern, however, is about other code that mistakenly adapted the same
pattern. After this fix its compilation will fail.
Yup. Any code that was using JNIEXPORT inappropriately will fail to compile.
All changes are incompatible changes.
But ... it's only a source incompatibility, only in native code, and trivial for users to fix.
(It would be even better if the semantics of JNIEXPORT were documented somewhere...)
Just like the JDK native code needs porting whenever a new release of gcc comes
out.
Martin's right, this is a concern that's beyond our scope and the problem is
easily solvable (the documentation issue aside..). Besides, it's going into 8
which will give it a chance to incubate before it gets out into the wild on
production systems, right?
(and I have a patch that will depend on Martin's patch coming shortly... so
anything to get this through will help :)
I did find at least a couple places in JavaFX 8.0 where it's being misused.
Those should be easy to clean up though, I can file an issue and handle it. I
haven't checked JavaFX 2.x yet, but that's not a concern until this gets
backported.
-DrD-