From 3b278a3bc71787b60444b98eaa414d4dfcd1eb47 Mon Sep 17 00:00:00 2001
From: ladyada <limor@ladyada.net>
Date: Sun, 25 Nov 2018 02:58:11 -0500
Subject: [PATCH] proper deiniting

---
 .../raspi_23/pulseio/PulseIn.py               | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)

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:
-- 
2.49.0