read even less of the map data, because we can't read part of some maps anymore and its data ggtracker doesn't need anyway.

Author: dsjoerg

sc2reader/objects.py

@@ -564,6 +564,10 @@ def __init__(self, contents):
         #: The map base height (what is that?). This value is 4096*Base Height in the editor (giving a decimal value).
         self.base_height = data.read_uint32()/4096
 
+        # Leave early so we dont barf. Turns out ggtracker doesnt need
+        # any of the map data thats loaded below.
+        return
+        
         #: Load screen type: 0 = default, 1 = custom
         self.load_screen_type = data.read_uint32()
 
@@ -627,10 +631,6 @@ def __init__(self, contents):
         #: The number of players enabled via the data editor
         self.player_count = data.read_uint32()
 
-        # Leave early so we dont barf. Turns out ggtracker doesnt need
-        # any of the map data thats loaded below.
-        return
-        
         #: A list of references to :class:`MapInfoPlayer` objects
         self.players = list()
         for i in range(self.player_count):

test_replays/test_all.py

@@ -432,6 +432,13 @@ def test_30(self):
       replay = sc2reader.load_replay("test_replays/3.0.0.38215/second.SC2Replay")
       replay = sc2reader.load_replay("test_replays/3.0.0.38215/third.SC2Replay")
 
+    def test_30_map(self):
+      for replayfilename in [
+              "test_replays/3.0.0.38215/third.SC2Replay",
+        ]:
+        factory = sc2reader.factories.SC2Factory()
+        replay =factory.load_replay(replayfilename,load_level=1,load_map= True)
+      
 
 class TestGameEngine(unittest.TestCase):
     class TestEvent(object):