From f56e36bd4a497643756ab737422eb356b963b27d Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Wed, 31 Oct 2012 16:41:28 +0100 Subject: Fix ProjectFloat (Bug 633): Adding missing offset of sliced buffer usage; reduce buffer usage (performance) in favor of float[]. Thomas De Bodt reported this error and provided the unit test. --- .../test/junit/jogl/acore/TestPMVMatrix02NEWT.java | 109 +++++++++++++++++++++ 1 file changed, 109 insertions(+) create mode 100644 src/test/com/jogamp/opengl/test/junit/jogl/acore/TestPMVMatrix02NEWT.java (limited to 'src/test/com/jogamp') diff --git a/src/test/com/jogamp/opengl/test/junit/jogl/acore/TestPMVMatrix02NEWT.java b/src/test/com/jogamp/opengl/test/junit/jogl/acore/TestPMVMatrix02NEWT.java new file mode 100644 index 000000000..4e959d460 --- /dev/null +++ b/src/test/com/jogamp/opengl/test/junit/jogl/acore/TestPMVMatrix02NEWT.java @@ -0,0 +1,109 @@ +/** + * Copyright 2012 JogAmp Community. All rights reserved. + * + * Redistribution and use in source and binary forms, with or without modification, are + * permitted provided that the following conditions are met: + * + * 1. Redistributions of source code must retain the above copyright notice, this list of + * conditions and the following disclaimer. + * + * 2. Redistributions in binary form must reproduce the above copyright notice, this list + * of conditions and the following disclaimer in the documentation and/or other materials + * provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY JogAmp Community ``AS IS'' AND ANY EXPRESS OR IMPLIED + * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND + * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL JogAmp Community OR + * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR + * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON + * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF + * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * The views and conclusions contained in the software and documentation are those of the + * authors and should not be interpreted as representing official policies, either expressed + * or implied, of JogAmp Community. + */ + +package com.jogamp.opengl.test.junit.jogl.acore; + +import org.junit.Before; +import org.junit.Test; + +import com.jogamp.opengl.util.PMVMatrix; + +import javax.media.opengl.fixedfunc.GLMatrixFunc; +import java.nio.FloatBuffer; + +import static org.junit.Assert.assertArrayEquals; + +/** + * @author Thomas De Bodt + */ +public class TestPMVMatrix02NEWT { + + private PMVMatrix fMat; + + @Before + public void setUp() throws Exception { + fMat = new PMVMatrix(); + } + + @Test + public void testLookAtNegZIsNoOp() throws Exception { + fMat.glMatrixMode(GLMatrixFunc.GL_MODELVIEW); + // Look towards -z + fMat.gluLookAt( + 0, 0, 0, + 0, 0, -1, + 0, 1, 0 + ); + FloatBuffer mvMatrix = fMat.glGetMvMatrixf(); + float[] mvMatrixArr = new float[16]; + mvMatrix.asReadOnlyBuffer().get(mvMatrixArr); + assertArrayEquals( + /** + * The 3 rows of the matrix (= the 3 columns of the array/buffer) should be: side, up, -forward. + */ + new float[] { + 1, 0, 0, 0, + 0, 1, 0, 0, + 0, 0, 1, 0, + 0, 0, 0, 1 + }, + mvMatrixArr, + 1e-6f + ); + } + @Test + public void testLookAtPosY() throws Exception { + fMat.glMatrixMode(GLMatrixFunc.GL_MODELVIEW); + // Look towards +y + fMat.gluLookAt( + 0, 0, 0, + 0, 1, 0, + 0, 0, 1 + ); + FloatBuffer mvMatrix = fMat.glGetMvMatrixf(); + float[] mvMatrixArr = new float[16]; + mvMatrix.asReadOnlyBuffer().get(mvMatrixArr); + assertArrayEquals( + /** + * The 3 rows of the matrix (= the 3 columns of the array/buffer) should be: side, up, -forward. + */ + new float[] { + 1, 0, 0, 0, + 0, 0, -1, 0, + 0, 1, 0, 0, + 0, 0, 0, 1 + }, + mvMatrixArr, + 1e-6f + ); + } + + public static void main(String args[]) { + org.junit.runner.JUnitCore.main(TestPMVMatrix02NEWT.class.getName()); + } +} -- cgit v1.2.3