aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSven Gothel <[email protected]>2015-03-11 20:42:09 +0100
committerSven Gothel <[email protected]>2015-03-11 20:42:09 +0100
commit99f14475993d127f1b927056b309477753563a02 (patch)
treebf9ca7c45e6735d8ffefadb7698a3b2d7e80ebd6
parenta622ffbced14e1e2fe797c82fc62c431ffb6949f (diff)
Resolve RenameJavaSymbol issues (GLUgl2 and CGL) using GLEmitter (-> DelegateImplementation) due to new GlueGen Semanticsv2.3.0
GlueGen would usually use the original native C method name for proc-address or direct call. GLEmitter however uses the renamed method name, assuming it is shorter and resulted from a 'RenameExtensionIntoCore' directive. GLUgl2 native usage was broken since 2.2.4 GLUgl2 and CGL use RenameJavaSymbol w/ the intention of delegation, hence resolve the issue using the new DelegateImplementation directive which preserves the original function name for the native call.
-rw-r--r--make/config/jogl/cgl-macosx.cfg9
-rw-r--r--make/config/jogl/glu-gl2.cfg14
-rw-r--r--make/scripts/tests.sh6
-rw-r--r--src/test/com/jogamp/opengl/test/junit/jogl/awt/text/TestAWTTextRendererUseVertexArrayBug464.java15
-rw-r--r--src/test/com/jogamp/opengl/test/junit/jogl/glu/TestBug694ScaleImageUnpackBufferSizeAWT.java1
5 files changed, 16 insertions, 29 deletions
diff --git a/make/config/jogl/cgl-macosx.cfg b/make/config/jogl/cgl-macosx.cfg
index 6ca0787b5..df39bdaab 100644
--- a/make/config/jogl/cgl-macosx.cfg
+++ b/make/config/jogl/cgl-macosx.cfg
@@ -37,12 +37,9 @@ Opaque long NSOpenGLLayer *
CustomCCode #include <machine/types.h>
CustomCCode #include "macosx-window-system.h"
-AccessControl createNSOpenGLLayerImpl PRIVATE
-AccessControl setNSOpenGLLayerEnabledImpl PRIVATE
-AccessControl releaseNSOpenGLLayerImpl PRIVATE
-RenameJavaSymbol createNSOpenGLLayer createNSOpenGLLayerImpl
-RenameJavaSymbol setNSOpenGLLayerEnabled setNSOpenGLLayerEnabledImpl
-RenameJavaSymbol releaseNSOpenGLLayer releaseNSOpenGLLayerImpl
+DelegateImplementation createNSOpenGLLayer createNSOpenGLLayerImpl
+DelegateImplementation setNSOpenGLLayerEnabled setNSOpenGLLayerEnabledImpl
+DelegateImplementation releaseNSOpenGLLayer releaseNSOpenGLLayerImpl
IncludeAs CustomJavaCode CGL cgl-macosx-CustomJavaCode.java
diff --git a/make/config/jogl/glu-gl2.cfg b/make/config/jogl/glu-gl2.cfg
index dea7992f7..54b9a15ad 100644
--- a/make/config/jogl/glu-gl2.cfg
+++ b/make/config/jogl/glu-gl2.cfg
@@ -69,12 +69,12 @@ AccessControl gluScaleImage PRIVATE
# Rename these methods in the implementing class
-RenameJavaSymbol gluBuild1DMipmapLevels gluBuild1DMipmapLevelsC
-RenameJavaSymbol gluBuild1DMipmaps gluBuild1DMipmapsC
-RenameJavaSymbol gluBuild2DMipmapLevels gluBuild2DMipmapLevelsC
-RenameJavaSymbol gluBuild2DMipmaps gluBuild2DMipmapsC
-RenameJavaSymbol gluBuild3DMipmapLevels gluBuild3DMipmapLevelsC
-RenameJavaSymbol gluBuild3DMipmaps gluBuild3DMipmapsC
-RenameJavaSymbol gluScaleImage gluScaleImageC
+DelegateImplementation gluBuild1DMipmapLevels gluBuild1DMipmapLevelsC
+DelegateImplementation gluBuild1DMipmaps gluBuild1DMipmapsC
+DelegateImplementation gluBuild2DMipmapLevels gluBuild2DMipmapLevelsC
+DelegateImplementation gluBuild2DMipmaps gluBuild2DMipmapsC
+DelegateImplementation gluBuild3DMipmapLevels gluBuild3DMipmapLevelsC
+DelegateImplementation gluBuild3DMipmaps gluBuild3DMipmapsC
+DelegateImplementation gluScaleImage gluScaleImageC
Include glu-common.cfg
diff --git a/make/scripts/tests.sh b/make/scripts/tests.sh
index 16bd57c1f..316262606 100644
--- a/make/scripts/tests.sh
+++ b/make/scripts/tests.sh
@@ -107,9 +107,9 @@ function jrun() {
#D_ARGS="-Djogamp.debug=all -Dnativewindow.debug=all -Djogl.debug=all -Dnewt.debug=all"
#D_ARGS="-Djogamp.debug.NativeLibrary=true -Djogamp.debug.JNILibLoader=true"
- #D_ARGS="-Djogamp.debug.ProcAddressHelper -Djogamp.debug.NativeLibrary -Djogamp.debug.NativeLibrary.Lookup -Djogamp.debug.JNILibLoader -Djogamp.debug.TempJarCache -Djogamp.debug.JarUtil"
#D_ARGS="-Djogl.debug.GLContext -Djogamp.debug.NativeLibrary -Djogamp.debug.JNILibLoader -Djogl.debug.DebugGL -Djogl.debug.GLDebugMessageHandler"
#D_ARGS="-Djogamp.debug.ProcAddressHelper -Djogamp.debug.NativeLibrary -Djogamp.debug.NativeLibrary.Lookup -Djogamp.debug.JNILibLoader -Djogamp.debug.TempJarCache -Djogamp.debug.JarUtil"
+ #D_ARGS="-Djogamp.debug.ProcAddressHelper -Djogamp.debug.NativeLibrary -Djogamp.debug.NativeLibrary.Lookup -Djogamp.debug.JNILibLoader -Djogamp.debug.TempJarCache -Djogamp.debug.JarUtil -Djogl.glu.nojava=true"
#D_ARGS="-Dnativewindow.debug=all -Djogl.debug=all -Dnewt.debug=all"
#D_ARGS="-Dnativewindow.debug=all -Dnewt.debug.Window"
@@ -505,7 +505,7 @@ function testawtswt() {
#testnoawt com.jogamp.opengl.test.junit.jogl.acore.TestGPUMemSec01NEWT $*
#testnoawt com.jogamp.opengl.test.junit.jogl.acore.TestGLException01NEWT $*
-testnoawt com.jogamp.opengl.test.junit.jogl.acore.TestMapBufferRead01NEWT $*
+#testnoawt com.jogamp.opengl.test.junit.jogl.acore.TestMapBufferRead01NEWT $*
#testnoawt com.jogamp.opengl.test.junit.jogl.demos.es2.newt.TestRedSquareES2NEWT $*
#testnoawt com.jogamp.opengl.test.junit.jogl.demos.es2.newt.TestGearsES2NEWT $*
@@ -689,7 +689,7 @@ testnoawt com.jogamp.opengl.test.junit.jogl.acore.TestMapBufferRead01NEWT $*
#testawt com.jogamp.opengl.test.junit.jogl.awt.TestIsRealizedConcurrency01AWT $*
#testawt com.jogamp.opengl.test.junit.jogl.awt.text.TestAWTTextRendererUseVertexArrayBug464
#testawt com.jogamp.opengl.test.junit.jogl.glu.TestBug463ScaleImageMemoryAWT $*
-#testawt com.jogamp.opengl.test.junit.jogl.glu.TestBug694ScaleImageUnpackBufferSizeAWT $*
+testawt com.jogamp.opengl.test.junit.jogl.glu.TestBug694ScaleImageUnpackBufferSizeAWT $*
#testawt com.jogamp.opengl.test.junit.jogl.glu.TestBug365TextureGenerateMipMaps $*
#testawt com.jogamp.opengl.test.junit.jogl.awt.TestGLCanvasAWTActionDeadlock00AWT $*
#testawt com.jogamp.opengl.test.junit.jogl.awt.TestGLCanvasAWTActionDeadlock01AWT $*
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/awt/text/TestAWTTextRendererUseVertexArrayBug464.java b/src/test/com/jogamp/opengl/test/junit/jogl/awt/text/TestAWTTextRendererUseVertexArrayBug464.java
index ff5eb6484..61d32f12f 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/awt/text/TestAWTTextRendererUseVertexArrayBug464.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/awt/text/TestAWTTextRendererUseVertexArrayBug464.java
@@ -32,7 +32,7 @@ import com.jogamp.opengl.GLProfile;
import com.jogamp.opengl.GLCapabilities;
import com.jogamp.opengl.awt.GLCanvas;
import com.jogamp.opengl.util.Animator;
-
+import com.jogamp.opengl.test.junit.jogl.acore.TestSharedContextVBOES2NEWT3;
import com.jogamp.opengl.test.junit.util.UITestCase;
import java.awt.Frame;
@@ -156,17 +156,6 @@ public class TestAWTTextRendererUseVertexArrayBug464 extends UITestCase {
}
public static void main(final String args[]) throws IOException {
- final String tstname = TestAWTTextRendererUseVertexArrayBug464.class.getName();
- org.apache.tools.ant.taskdefs.optional.junit.JUnitTestRunner.main(new String[] {
- tstname,
- "filtertrace=true",
- "haltOnError=false",
- "haltOnFailure=false",
- "showoutput=true",
- "outputtoformatters=true",
- "logfailedtests=true",
- "logtestlistenerevents=true",
- "formatter=org.apache.tools.ant.taskdefs.optional.junit.PlainJUnitResultFormatter",
- "formatter=org.apache.tools.ant.taskdefs.optional.junit.XMLJUnitResultFormatter,TEST-"+tstname+".xml" } );
+ org.junit.runner.JUnitCore.main(TestAWTTextRendererUseVertexArrayBug464.class.getName());
}
}
diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/glu/TestBug694ScaleImageUnpackBufferSizeAWT.java b/src/test/com/jogamp/opengl/test/junit/jogl/glu/TestBug694ScaleImageUnpackBufferSizeAWT.java
index 4556a6806..c0e6e05d2 100644
--- a/src/test/com/jogamp/opengl/test/junit/jogl/glu/TestBug694ScaleImageUnpackBufferSizeAWT.java
+++ b/src/test/com/jogamp/opengl/test/junit/jogl/glu/TestBug694ScaleImageUnpackBufferSizeAWT.java
@@ -67,6 +67,7 @@ public class TestBug694ScaleImageUnpackBufferSizeAWT extends UITestCase implemen
final GL gl = drawable.getGL();
final GLU glu = GLU.createGLU(gl);
+ System.err.println("Using GLU "+glu.getClass().getName());
testGLUScaleImage(gl, glu, 0); // default 4
testGLUScaleImage(gl, glu, 1);
testGLUScaleImage(gl, glu, 4);