revpi/revpimodio2
1
0
Fork 0
mirror of https://github.com/naruxde/revpimodio2.git synced 2025-11-08 13:53:53 +01:00
Code Issues Packages Releases Wiki Activity

fix: Context manager crashes in monitoring mode

Browse Source 
The context managers tried to write to the process image in
monitoring=True mode. This is not allowed and the application crashed
 with an error.

Signed-off-by: Sven Sager <akira@narux.de>
This commit is contained in:
Sven Sager
2023-12-01 15:58:28 +01:00
parent ad4a1fcd38
commit 41a09c4f17
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/revpimodio2/io.py
View File

@@ -115,7 +115,8 @@ class IOList(object):
self.__modio._imgwriter.newdata.clear() self.__modio._imgwriter.newdata.clear()
# Write outputs on devices without autorefresh # Write outputs on devices without autorefresh
self.__modio.writeprocimg() if not self.__modio._monitoring:
self.__modio.writeprocimg()
if self.__modio._imgwriter.is_alive(): if self.__modio._imgwriter.is_alive():
# Wait until imgwriter has written outputs # Wait until imgwriter has written outputs

3
src/revpimodio2/modio.py
View File

@@ -218,7 +218,8 @@ class RevPiModIO(object):
return self return self
def __exit__(self, exc_type, exc_val, exc_tb): def __exit__(self, exc_type, exc_val, exc_tb):
self.writeprocimg() if not self._monitoring:
self.writeprocimg()
self.exit(full=True) self.exit(full=True)
self._looprunning = False self._looprunning = False
self._context_manager = False self._context_manager = False