From: ladyada <limor@ladyada.net>
Date: Sun, 25 Nov 2018 07:58:11 +0000 (-0500)
Subject: proper deiniting
X-Git-Tag: 0.4.0~3^2~3
X-Git-Url: https://git.ayoreis.com/Adafruit_Blinka-hackapet.git/commitdiff_plain/3b278a3bc71787b60444b98eaa414d4dfcd1eb47

proper deiniting
---

diff --git a/src/adafruit_blinka/microcontroller/raspi_23/pulseio/PulseIn.py b/src/adafruit_blinka/microcontroller/raspi_23/pulseio/PulseIn.py
index ef77b74..2cda647 100644
--- a/src/adafruit_blinka/microcontroller/raspi_23/pulseio/PulseIn.py
+++ b/src/adafruit_blinka/microcontroller/raspi_23/pulseio/PulseIn.py
@@ -47,8 +47,8 @@ class PulseIn:
         if DEBUG:
             print(cmd)
         
-        self.process = subprocess.Popen(cmd)
-        procs.append(self.process)
+        self._process = subprocess.Popen(cmd)
+        procs.append(self._process)
 
         # wait for it to start up
         if DEBUG:
@@ -58,18 +58,18 @@ class PulseIn:
             raise RuntimeError("Could not establish message queue with subprocess")
 
 
-    def __deinit__(self):
-        print("deinit")
-        
-    # TODO: this doesnt work?
+    def deinit(self):
+        # Clean up after ourselves
+        self._process.terminate()
+        procs.remove(self._process)
+        self._mq.remove()
+        queues.remove(self._mq)
+
     def __enter__(self):
-        print("enter")
-        pass # no-op
+        return self
 
-    # TODO: this doesnt work?
     def __exit__(self, exc_type, exc_value, tb):
-        print("exit")
-        #self.process.kill()
+        self.deinit()
 
     def resume(self, trigger_duration=0):
         if trigger_duration != 0: