commit d07b976d59b053eaf12539c474f4f94243985b84
author Syoyo Fujita <syoyo@lighttransport.com> Wed Apr 28 19:15:16 2021 +0900
committer Syoyo Fujita <syoyo@lighttransport.com> Wed Apr 28 19:15:16 2021 +0900
tree 417bd3351d794b6a35b5470fb84b272e5727996a
parent 08a7dd8dadd303e4cb29f7c1a4525047e04cc5c0

Do not write negative `skeleton` id(skeleton is not assigned).
Also ensure `inverseBindMatrices` should not be -2 or other negative values.

tiny_gltf.h

diff --git a/tiny_gltf.h b/tiny_gltf.h
index 15d738c..0aa9b76 100644
--- a/tiny_gltf.h
+++ b/tiny_gltf.h
@@ -7243,11 +7243,17 @@
 }
 
 static void SerializeGltfSkin(Skin &skin, json &o) {
-  if (skin.inverseBindMatrices != -1)
-    SerializeNumberProperty("inverseBindMatrices", skin.inverseBindMatrices, o);
-
+  // required
   SerializeNumberArrayProperty<int>("joints", skin.joints, o);
-  SerializeNumberProperty("skeleton", skin.skeleton, o);
+
+  if (skin.inverseBindMatrices >= 0) {
+    SerializeNumberProperty("inverseBindMatrices", skin.inverseBindMatrices, o);
+  }
+
+  if (skin.skeleton >= 0) {
+    SerializeNumberProperty("skeleton", skin.skeleton, o);
+  }
+
   if (skin.name.size()) {
     SerializeStringProperty("name", skin.name, o);
   }