Attachment 650 Details for Bug 1078 – patch (part 1), with the changes in GLRendererQuirks and GLContextImpl

[patch] patch (part 1), with the changes in GLRendererQuirks and GLContextImpl

bug1078_patch1.txt (text/plain), 2.95 KB, created by Julien Gouesse on 2014-09-30 17:52:03 CEST

(hide)

Description:

Filename:

MIME Type:

Creator: Julien Gouesse

Created: 2014-09-30 17:52:03 CEST

Size: 2.95 KB

patch

obsolete

>diff --git a/src/jogl/classes/com/jogamp/opengl/GLRendererQuirks.java b/src/jogl/classes/com/jogamp/opengl/GLRendererQuirks.java
>index f81d2f2..2c29885 100644
>--- a/src/jogl/classes/com/jogamp/opengl/GLRendererQuirks.java
>+++ b/src/jogl/classes/com/jogamp/opengl/GLRendererQuirks.java
>@@ -341,8 +341,24 @@
>      */
>     public static final int BuggyColorRenderbuffer  = 18;
> 
>+    /**
>+     * Some Intel drivers under Windows wrongly claims to support pbuffers 
>+     * with accumulation buffers. Then, the creation of a pbuffer fails:
>+     * javax.media.opengl.GLException: pbuffer creation error: Couldn't find a suitable pixel format
>+     * <p>
>+     * Appears on:
>+     * <ul>
>+     *   <li>GL_VENDOR       Intel</li>
>+     *   <li>GL_RENDERER     Intel Bear Lake B</li>
>+     *   <li>GL_VERSION      1.4.0 - Build 8.14.10.1930</li>
>+     * </ul>
>+     * 
>+     * </p>
>+     */
>+    public static final int NoPBufferWithAccum = 19;
>+    
>     /** Number of quirks known. */
>-    public static final int COUNT = 19;
>+    public static final int COUNT = 20;
> 
>     private static final String[] _names = new String[] { "NoDoubleBufferedPBuffer", "NoDoubleBufferedBitmap", "NoSetSwapInterval",
>                                                           "NoOffscreenBitmap", "NoSetSwapIntervalPostRetarget", "GLSLBuggyDiscard",
>@@ -350,7 +366,7 @@
>                                                           "NeedCurrCtx4ARBPixFmtQueries", "NeedCurrCtx4ARBCreateContext",
>                                                           "NoFullFBOSupport", "GLSLNonCompliant", "GL4NeedsGL3Request",
>                                                           "GLSharedContextBuggy", "GLES3ViaEGLES2Config", "SingletonEGLDisplayOnly",
>-                                                          "NoMultiSamplingBuffers", "BuggyColorRenderbuffer"
>+                                                          "NoMultiSamplingBuffers", "BuggyColorRenderbuffer", "NoPBufferWithAccum"
>                                                         };
> 
>     private static final IdentityHashMap<String, GLRendererQuirks> stickyDeviceQuirks = new IdentityHashMap<String, GLRendererQuirks>();
>diff --git a/src/jogl/classes/jogamp/opengl/GLContextImpl.java b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
>index 01478a4..f1dee79 100644
>--- a/src/jogl/classes/jogamp/opengl/GLContextImpl.java
>+++ b/src/jogl/classes/jogamp/opengl/GLContextImpl.java
>@@ -1841,6 +1841,13 @@
>                 quirks.addQuirk( quirk );
>             }
>         }
>+        if (glVendor.equals("Intel") && glRenderer.equals("Intel Bear Lake B")) {
>+        	final int quirk = GLRendererQuirks.NoPBufferWithAccum;
>+        	if(DEBUG) {
>+                System.err.println("Quirk: "+GLRendererQuirks.toString(quirk)+": cause: OS "+Platform.getOSType()+", [Vendor "+glVendor+" and Renderer "+glRenderer+"]");
>+            }
>+        	quirks.addQuirk( quirk );
>+        }
>     } else if( Platform.OSType.ANDROID == Platform.getOSType() ) {
>         //
>         // ANDROID

Actions: View | Diff

Attachments on bug 1078: 640 | 641 | 642 | 643 | 644 | 645 | 646 | 647 | 648 | 649 | 650 | 651 | 652 | 654 | 655