glue: check the values returned by JNIEnv::GetMethodID()

stephengold · stephengold · commit 2b4a75ee8fa7 · 2026-02-26T16:54:44.000-08:00
diff --git a/src/main/native/glue/cu/CustomBodyActivationListener.cpp b/src/main/native/glue/cu/CustomBodyActivationListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -50,9 +50,11 @@ class CustomBodyActivationListener : BodyActivationListener {
         EXCEPTION_CHECK(pEnv)
 
         mActivatedMethodId = pEnv->GetMethodID(clss, "onBodyActivated", "(IJ)V");
+        JPH_ASSERT(NULL != mActivatedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mDeactivatedMethodId = pEnv->GetMethodID(clss, "onBodyDeactivated", "(IJ)V");
+        JPH_ASSERT(NULL != mDeactivatedMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCastRayCollector.cpp b/src/main/native/glue/cu/CustomCastRayCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCastRayCollector : CastRayCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCastShapeBodyCollector.cpp b/src/main/native/glue/cu/CustomCastShapeBodyCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2025 Stephen Gold
+Copyright (c) 2025-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCastShapeBodyCollector : CastShapeBodyCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCastShapeCollector.cpp b/src/main/native/glue/cu/CustomCastShapeCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCastShapeCollector : CastShapeCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCharacterContactListener.cpp b/src/main/native/glue/cu/CustomCharacterContactListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -58,46 +58,57 @@ class CustomCharacterContactListener : CharacterContactListener {
 
         mAddedMethodId = pEnv->GetMethodID(
                 clss, "onContactAdded", "(JIIDDDFFFJ)V");
+        JPH_ASSERT(NULL != mAddedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mAdjustMethodId = pEnv->GetMethodID(
                 clss, "onAdjustBodyVelocity", "(JJ[F)V");
+        JPH_ASSERT(NULL != mAdjustMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mCcAddedMethodId = pEnv->GetMethodID(
                 clss, "onCharacterContactAdded", "(JJIDDDFFFJ)V");
+        JPH_ASSERT(NULL != mCcAddedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mCcPersistedMethodId = pEnv->GetMethodID(
                 clss, "onCharacterContactPersisted", "(JJIDDDFFFJ)V");
+        JPH_ASSERT(NULL != mCcPersistedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mCcRemovedMethodId = pEnv->GetMethodID(
                 clss, "onCharacterContactRemoved", "(JII)V");
+        JPH_ASSERT(NULL != mCcRemovedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mCcSolveMethodId = pEnv->GetMethodID(
                 clss, "onCharacterContactSolve", "(JJIDDDFFFFFFJFFF[F)V");
+        JPH_ASSERT(NULL != mCcSolveMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mCcValidateMethodId = pEnv->GetMethodID(
                 clss, "onCharacterContactValidate", "(JJI)Z");
+        JPH_ASSERT(NULL != mCcValidateMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mPersistedMethodId = pEnv->GetMethodID(
                 clss, "onContactPersisted", "(JIIDDDFFFJ)V");
+        JPH_ASSERT(NULL != mPersistedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mRemovedMethodId = pEnv->GetMethodID(
                 clss, "onContactRemoved", "(JII)V");
+        JPH_ASSERT(NULL != mRemovedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mSolveMethodId = pEnv->GetMethodID(
                 clss, "onContactSolve", "(JIIDDDFFFFFFJFFF[F)V");
+        JPH_ASSERT(NULL != mSolveMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mValidateMethodId = pEnv->GetMethodID(
                 clss, "onContactValidate", "(JII)Z");
+        JPH_ASSERT(NULL != mValidateMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCollidePointCollector.cpp b/src/main/native/glue/cu/CustomCollidePointCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCollidePointCollector : CollidePointCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCollideShapeBodyCollector.cpp b/src/main/native/glue/cu/CustomCollideShapeBodyCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCollideShapeBodyCollector : CollideShapeBodyCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(I)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomCollideShapeCollector.cpp b/src/main/native/glue/cu/CustomCollideShapeCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomCollideShapeCollector : CollideShapeCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomContactListener.cpp b/src/main/native/glue/cu/CustomContactListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -50,15 +50,19 @@ class CustomContactListener : ContactListener {
         EXCEPTION_CHECK(pEnv)
 
         mAddedMethodId = pEnv->GetMethodID(clss, "onContactAdded", "(JJJJ)V");
+        JPH_ASSERT(NULL != mAddedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mPersistedMethodId = pEnv->GetMethodID(clss, "onContactPersisted", "(JJJJ)V");
+        JPH_ASSERT(NULL != mPersistedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mRemovedMethodId = pEnv->GetMethodID(clss, "onContactRemoved", "(J)V");
+        JPH_ASSERT(NULL != mRemovedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mValidateMethodId = pEnv->GetMethodID(clss, "onContactValidate", "(JJDDDJ)I");
+        JPH_ASSERT(NULL != mValidateMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomDebugRendererSimple.cpp b/src/main/native/glue/cu/CustomDebugRendererSimple.cpp
@@ -50,14 +50,17 @@ class CustomDebugRendererSimple : public DebugRendererSimple {
         EXCEPTION_CHECK(pEnv)
 
         mDrawLineId = pEnv->GetMethodID(clss, "drawLine", "(DDDDDDI)V");
+        JPH_ASSERT(NULL != mDrawLineId);
         EXCEPTION_CHECK(pEnv)
 
         mDrawTextId = pEnv->GetMethodID(
                 clss, "drawText3d", "(DDDLjava/lang/String;IF)V");
+        JPH_ASSERT(NULL != mDrawTextId);
         EXCEPTION_CHECK(pEnv)
 
         mDrawTriangleId
                 = pEnv->GetMethodID(clss, "drawTriangle", "(DDDDDDDDDII)V");
+        JPH_ASSERT(NULL != mDrawTriangleId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomPhysicsStepListener.cpp b/src/main/native/glue/cu/CustomPhysicsStepListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomPhysicsStepListener : PhysicsStepListener {
         EXCEPTION_CHECK(pEnv)
 
         mStepMethodId = pEnv->GetMethodID(clss, "onStep", "(J)V");
+        JPH_ASSERT(NULL != mStepMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomRayCastBodyCollector.cpp b/src/main/native/glue/cu/CustomRayCastBodyCollector.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -47,6 +47,7 @@ class CustomRayCastBodyCollector : RayCastBodyCollector {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHit", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/cu/CustomSoftBodyContactListener.cpp b/src/main/native/glue/cu/CustomSoftBodyContactListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2025 Stephen Gold
+Copyright (c) 2025-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -49,9 +49,11 @@ class CustomSoftBodyContactListener : SoftBodyContactListener {
         EXCEPTION_CHECK(pEnv)
 
         mAddedMethodId = pEnv->GetMethodID(clss, "onSoftBodyContactAdded", "(JJ)V");
+        JPH_ASSERT(NULL != mAddedMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mValidateMethodId = pEnv->GetMethodID(clss, "onSoftBodyContactValidate", "(JJJ)I");
+        JPH_ASSERT(NULL != mValidateMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/d/Decomposer.cpp b/src/main/native/glue/d/Decomposer.cpp
@@ -51,10 +51,12 @@ class Decomposer : public IVHACD::IUserCallback, public IVHACD::IUserLogger {
         EXCEPTION_CHECK(pEnv)
 
         mAddMethodId = pEnv->GetMethodID(clss, "addHull", "(J)V");
+        JPH_ASSERT(NULL != mAddMethodId);
         EXCEPTION_CHECK(pEnv)
 
         mUpdateMethodId = pEnv->GetMethodID(
                 clss, "update", "(DDDLjava/lang/String;Ljava/lang/String;)V");
+        JPH_ASSERT(NULL != mUpdateMethodId);
         EXCEPTION_CHECK(pEnv)
     }
 
diff --git a/src/main/native/glue/f/FilteredContactListener.cpp b/src/main/native/glue/f/FilteredContactListener.cpp
@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2025 Stephen Gold
+Copyright (c) 2025-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -68,14 +68,18 @@ class FilteredContactListener : ContactListener {
                 "com/github/stephengold/joltjni/FilteredContactListener");
         EXCEPTION_CHECK(pEnv)
         mAddedMethodId = pEnv->GetMethodID(clss, "onContactAdded", "(JJJJ)V");
+        JPH_ASSERT(NULL != mAddedMethodId);
         EXCEPTION_CHECK(pEnv)
         mPersistedMethodId
                 = pEnv->GetMethodID(clss, "onContactPersisted", "(JJJJ)V");
+        JPH_ASSERT(NULL != mPersistedMethodId);
         EXCEPTION_CHECK(pEnv)
         mRemovedMethodId = pEnv->GetMethodID(clss, "onContactRemoved", "(J)V");
+        JPH_ASSERT(NULL != mRemovedMethodId);
         EXCEPTION_CHECK(pEnv)
         mValidateMethodId
                 = pEnv->GetMethodID(clss, "onContactValidate", "(JJDDDJ)I");
+        JPH_ASSERT(NULL != mValidateMethodId);
         EXCEPTION_CHECK(pEnv)
     }
     void OnContactAdded(const Body& inBody1, const Body& inBody2,
diff --git a/src/main/native/glue/j/Jolt.cpp b/src/main/native/glue/j/Jolt.cpp
@@ -582,9 +582,11 @@ JNIEXPORT void JNICALL Java_com_github_stephengold_joltjni_Jolt_installJavaTrace
     const jclass clss = pEnv->FindClass("java/io/PrintStream");
     EXCEPTION_CHECK(pEnv)
     gFlushMethodId = pEnv->GetMethodID(clss, "flush", "()V");
+    JPH_ASSERT(NULL != gFlushMethodId);
     EXCEPTION_CHECK(pEnv)
     gPrintMethodId
             = pEnv->GetMethodID(clss, "println", "(Ljava/lang/String;)V");
+    JPH_ASSERT(NULL != gPrintMethodId);
     EXCEPTION_CHECK(pEnv)
     //
     Trace = JavaTrace;
