From 976d45a2309784820f295b699c9089bdbbf5d314 Mon Sep 17 00:00:00 2001 From: Stephen Gold Date: Wed, 3 Oct 2018 14:52:09 -0700 Subject: [PATCH] test and fix for issue #928: disabled native Bullet's QuickProf --- jme3-bullet-native/build.gradle | 1 + .../java/jme3test/bullet/TestIssue928.java | 80 +++++++++++++++++++ 2 files changed, 81 insertions(+) create mode 100644 jme3-examples/src/main/java/jme3test/bullet/TestIssue928.java diff --git a/jme3-bullet-native/build.gradle b/jme3-bullet-native/build.gradle index a08bb81a4..55273f09e 100644 --- a/jme3-bullet-native/build.gradle +++ b/jme3-bullet-native/build.gradle @@ -87,6 +87,7 @@ model { return } + cppCompiler.define('BT_NO_PROFILE') if (toolChain in VisualCpp) { cppCompiler.args "/I$javaHome\\include" } else{ diff --git a/jme3-examples/src/main/java/jme3test/bullet/TestIssue928.java b/jme3-examples/src/main/java/jme3test/bullet/TestIssue928.java new file mode 100644 index 000000000..886fb8259 --- /dev/null +++ b/jme3-examples/src/main/java/jme3test/bullet/TestIssue928.java @@ -0,0 +1,80 @@ +/* + * Copyright (c) 2018 jMonkeyEngine + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are + * met: + * + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * + * * 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. + * + * * Neither the name of 'jMonkeyEngine' nor the names of its contributors + * may be used to endorse or promote products derived from this software + * without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "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 THE COPYRIGHT OWNER 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. + */ +package jme3test.bullet; + +import com.jme3.app.SimpleApplication; +import com.jme3.bullet.BulletAppState; +import com.jme3.bullet.BulletAppState.ThreadingType; + +/** + * Test case for JME issue #928: crash after 64 attached and detached + * BulletAppStates with parallel threading. The bug existed in Native Bullet + * only. + *

+ * If successful, no crash will occur. + */ +public class TestIssue928 extends SimpleApplication { + // ************************************************************************* + // new methods exposed + + public static void main(String[] args) { + TestIssue928 app = new TestIssue928(); + app.start(); + } + + int count = 0; + int frame = 0; + BulletAppState bulletAppState; + // ************************************************************************* + // SimpleApplication methods + + @Override + public void simpleInitApp() { + } + + @Override + public void simpleUpdate(float tpf) { + if (frame % 4 == 0) { + System.out.println(++count); + bulletAppState = new BulletAppState(); + bulletAppState.setThreadingType(ThreadingType.PARALLEL); + stateManager.attach(bulletAppState); + } else if (frame % 4 == 2) { + stateManager.detach(bulletAppState); + } + + frame++; + if (count == 70) { + System.exit(0); + } + } +}