From: Angus Gratton Date: Mon, 21 May 2018 07:33:10 +0000 (+1000) Subject: gen_esp_err_to_name.py: Use normal file I/O instead of mmap() X-Git-Tag: v3.1-beta1~99^2 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b14bbd4205f7c300bd8a5aea0a104d2d40a1ddce;p=esp-idf gen_esp_err_to_name.py: Use normal file I/O instead of mmap() mmap is different on Python for Windows --- diff --git a/tools/gen_esp_err_to_name.py b/tools/gen_esp_err_to_name.py index 1dfe15bab5..7281ed5511 100755 --- a/tools/gen_esp_err_to_name.py +++ b/tools/gen_esp_err_to_name.py @@ -16,7 +16,6 @@ import os import argparse -import mmap import re import fnmatch import string @@ -277,19 +276,14 @@ def main(): idf_path = os.path.relpath(full_path, os.environ['IDF_PATH']) if idf_path in ignore_files: continue - with open(full_path, "r+b") as f: - try: - map = mmap.mmap(f.fileno(), 0, prot=mmap.ACCESS_READ) - except ValueError: - pass # An empty file cannot be mmaped - else: - for line in iter(map.readline, ""): - # match also ESP_OK and ESP_FAIL because some of ESP_ERRs are referencing them - if re.match(r"\s*#define\s+(ESP_ERR_|ESP_OK|ESP_FAIL)", line): - try: - process(str.strip(line), idf_path) - except InputError as e: - print (e) + with open(full_path, "r+") as f: + for line in f: + # match also ESP_OK and ESP_FAIL because some of ESP_ERRs are referencing them + if re.match(r"\s*#define\s+(ESP_ERR_|ESP_OK|ESP_FAIL)", line): + try: + process(str.strip(line), idf_path) + except InputError as e: + print (e) process_remaining_errors()