]> granicus.if.org Git - esp-idf/commitdiff
make size: Fix map file parsing error on Windows
authorAngus Gratton <angus@espressif.com>
Thu, 29 Jun 2017 02:54:13 +0000 (12:54 +1000)
committerAngus Gratton <gus@projectgus.com>
Thu, 29 Jun 2017 02:54:13 +0000 (12:54 +1000)
Closes https://github.com/espressif/esp-idf/issues/739

tools/idf_size.py

index 1116af362eeaeb996e0183807be104dbfe4f831f..aa1d76759993fc4644b073a6096272b573719a62 100755 (executable)
@@ -57,12 +57,12 @@ def load_memory_config(map_file):
     """ Memory Configuration section is the total size of each output section """
     result = {}
     scan_to_header(map_file, "Memory Configuration")
-    RE_MEMORY_SECTION = r"(?P<name>[^ ]+) +0x(?P<origin>[\da-f]+) 0x(?P<length>[\da-f]+)"
+    RE_MEMORY_SECTION = r"(?P<name>[^ ]+) +0x(?P<origin>[\da-f]+) +0x(?P<length>[\da-f]+)"
     for line in map_file:
         m = re.match(RE_MEMORY_SECTION, line)
         if m is None:
             if len(result) == 0:
-                continue  # whitespace or a header
+                continue  # whitespace or a header, before the content we want
             else:
                 return result  # we're at the end of the Memory Configuration
         section = {
@@ -72,6 +72,7 @@ def load_memory_config(map_file):
         }
         if section["name"] != "*default*":
             result[section["name"]] = section
+    raise RuntimeError("End of file while scanning memory configuration?")
 
 def load_sections(map_file):
     """ Load section size information from the MAP file.