Crash_Override/OVMS3-idf
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge branch 'bugfix/idf_monitor_windows_console' into 'master'

Browse source 
idf_monitor: Ignore OS-level errors writing to Windows Console

Closes IDF-810

See merge request espressif/esp-idf!5607
This commit is contained in:
Angus Gratton 2020-01-13 12:13:42 +08:00
parent a5a5c318f9 2e43d107b4
commit 0929c91d1a
1 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
tools/idf_monitor.py
View file

@ -899,10 +899,13 @@ if os.name == 'nt':
self.output.write(data.decode())
else:
self.output.write(data)
except IOError:
except (IOError, OSError):
# Windows 10 bug since the Fall Creators Update, sometimes writing to console randomly throws
# an exception (however, the character is still written to the screen)
# Ref https://github.com/espressif/esp-idf/issues/1136
# Ref https://github.com/espressif/esp-idf/issues/1163
#
# Also possible for Windows to throw an OSError error if the data is invalid for the console
# (garbage bytes, etc)
pass
def write(self, data):
@ -939,7 +942,12 @@ if os.name == 'nt':
self.matched = b''
def flush(self):
self.output.flush()
try:
self.output.flush()
except OSError:
# Account for Windows Console refusing to accept garbage bytes (serial noise, etc)
pass
if __name__ == "__main__":
main()