Many msh files (e.g. those in BFX) have multiple models assigned to the same index (MNDX). Indices should be linked only to the first model that uses them to ensure proper skinning.

addons/io_scene_swbf_msh/__init__.py

@@ -1,7 +1,7 @@
 bl_info = {
     'name': 'SWBF .msh Import-Export',
     'author': 'Will Snyder, SleepKiller',
-    "version": (1, 2, 1),
+    "version": (1, 0, 0),
     'blender': (2, 80, 0),
     'location': 'File > Import-Export',
     'description': 'Export as SWBF .msh file',

addons/io_scene_swbf_msh/chunked_file_reader.py

@@ -138,9 +138,6 @@ class Reader:
     def how_much_left(self, pos):
         return self.end_pos - pos
 
-    def bytes_remaining(self):
-        return self.end_pos - self.file.tell()
-
     def skip_until(self, header):
         while (self.could_have_child() and header not in self.peak_next_header()):
             self.skip_bytes(1)

addons/io_scene_swbf_msh/msh_scene_read.py

@@ -165,23 +165,19 @@ def _read_matd(matd: Reader) -> Material:
 
         elif next_header == "TX0D":
             with matd.read_child() as tx0d:
-                if tx0d.bytes_remaining() > 0:
+                mat.texture0 = tx0d.read_string()
-                    mat.texture0 = tx0d.read_string()
 
         elif next_header == "TX1D":
             with matd.read_child() as tx1d:
-                if tx1d.bytes_remaining() > 0:
+                mat.texture1 = tx1d.read_string()
-                    mat.texture1 = tx1d.read_string()
 
         elif next_header == "TX2D":
             with matd.read_child() as tx2d:
-                if tx2d.bytes_remaining() > 0:
+                mat.texture2 = tx2d.read_string()
-                    mat.texture2 = tx2d.read_string()
 
         elif next_header == "TX3D":
             with matd.read_child() as tx3d:
-                if tx3d.bytes_remaining() > 0:
+                mat.texture3 = tx3d.read_string()
-                    mat.texture3 = tx3d.read_string()
 
         else:
             matd.skip_bytes(1)