From b15eb3df080ed0d716237aa271221f8ed4b25d5e Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Tue, 22 Feb 2011 07:45:49 +0100 Subject: JOGL/X11/AWT: No more need for seperate X11 Display instance if available --- .../x11/glx/awt/X11AWTGLXGraphicsConfigurationFactory.java | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'src/jogl/classes/jogamp') diff --git a/src/jogl/classes/jogamp/opengl/x11/glx/awt/X11AWTGLXGraphicsConfigurationFactory.java b/src/jogl/classes/jogamp/opengl/x11/glx/awt/X11AWTGLXGraphicsConfigurationFactory.java index 55f1f034b..ee7dd280e 100644 --- a/src/jogl/classes/jogamp/opengl/x11/glx/awt/X11AWTGLXGraphicsConfigurationFactory.java +++ b/src/jogl/classes/jogamp/opengl/x11/glx/awt/X11AWTGLXGraphicsConfigurationFactory.java @@ -96,13 +96,12 @@ public class X11AWTGLXGraphicsConfigurationFactory extends GLGraphicsConfigurati System.err.println(Thread.currentThread().getName() + " - X11AWTGLXGraphicsConfigurationFactory: using AWT X11 display 0x"+Long.toHexString(displayHandle)); } /** - * May cause an exception on NVidia X11 Display destruction, - * when destruction happen immediately after construction, around 10ms. - * However, the own Display handle is required for AMD gpus. + * Using the AWT display handle works fine with NVidia and AMD drivers today 2011-02-22, + * hence no need for our own display instance anymore. + String name = X11Util.XDisplayString(displayHandle); + displayHandle = X11Util.createDisplay(name); + owner = true; */ - String name = X11Util.XDisplayString(displayHandle); - displayHandle = X11Util.createDisplay(name); - owner = true; } ((AWTGraphicsDevice)awtScreen.getDevice()).setSubType(NativeWindowFactory.TYPE_X11, displayHandle); X11GraphicsDevice x11Device = new X11GraphicsDevice(displayHandle, AbstractGraphicsDevice.DEFAULT_UNIT); -- cgit v1.2.3