Don't forget to pthread_mutex_destroy()

This was a bit of an unfortunate omission. It meant we were leaking our mutexes. Oops.
postmates · Dec 18, 2015 · 6634a2a · 6634a2a
1 parent 4499064
commit 6634a2a
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/PMKVObserver/KVObserver.m b/PMKVObserver/KVObserver.m
@@ -110,6 +110,13 @@ - (instancetype)init {
     @throw [NSException exceptionWithName:NSInternalInconsistencyException reason:@"-[PMKVObserver init] is not available" userInfo:nil];
 }
 
+- (void)dealloc {
+    int retval = pthread_mutex_destroy(&_mutex);
+    if (__builtin_expect(retval, 0) != 0) {
+        NSLog(@"PMKVObserver: pthread_mutex_destroy: %s", strerror(retval));
+    }
+}
+
 - (void)cancel {
     [self cancel:NO];
 }