Review request for 8157002: Toggle gtk version if SWT used via FXCanvas
tom.schindl at bestsolution.at
Wed Nov 23 16:31:34 UTC 2016
Just replied to Semyon and I think the change is fine
On 23.11.16 16:37, Kevin Rushforth wrote:
> Hi Tom,
> Would you have time to help us test this?
> -- Kevin
> Tom Schindl wrote:
>> From looking at the code I doubt your fix will work reliably in a
>> OSGi-Environment who is the Main deployment scenario for SWT and hence
>> For sure you won't find the SWT-Library on the SystemClassloader and
>> whether you find it on the Thread-ContextClassloader is just gambling!
>> The only area you for sure can detect the swt-library are the
>> swt-fx.jar-Classes because those are guaranteed to be loaded with a
>> classloader who can look up SWT-Libary classes like
>> BTW: Is the statement below really correct?
>> .... in JFX embeded into SWT scenarios JFX loads GTK primarily ...
>> If you create an instance of FXCanvas SWT must have been loaded already
>> (You need to pass a parent Composite) so the native gtk-libs are there
>> already loaded.
>> On 22.11.16 14:51, Semyon Sadetsky wrote:
>>> Hello Kevin & David,
>>> Please review the fix for jfx9:
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8157002
>>> webrev: http://cr.openjdk.java.net/~ssadetsky/8157002/webrev.00/
>>> In JFX embeded into SWT scenarios JFX loads GTK primarily. So SWT GTK
>>> version cannot be detected using the check for the loaded native library
>>> The fix proposes a way to detect if GTK version of swt.jar library is
>>> available on the classpath and tries to get the GTK version from SWT lib
>>> internal java classes.
Thomas Schindl, CTO
BestSolution.at EDV Systemhaus GmbH
Eduard-Bodem-Gasse 5-7, A-6020 Innsbruck
Reg. Nr. FN 222302s am Firmenbuchgericht Innsbruck
More information about the openjfx-dev