Removed destructors because they aren't reliable

src/Lock/FlockLock.php

@@ -108,23 +108,6 @@ public function __clone()
         $this->files = array();
     }
 
-    /**
-     * Try to release any obtained locks when object is destroyed
-     *
-     * This is a safe guard for cases when your php script dies unexpectedly.
-     * It's not guaranteed it will work either.
-     *
-     * You should not depend on __destruct() to release your locks,
-     * instead release them with `$released = $this->releaseLock()`A
-     * and check `$released` if lock was properly released
-     */
-    public function __destruct()
-    {
-        while (null !== $file = array_pop($this->files)) {
-            fclose($file);
-        }
-    }
-
     /**
      * Check if lock is locked
      *

src/Lock/LockAbstract.php

@@ -9,8 +9,6 @@
  */
 namespace NinjaMutex\Lock;
 
-use NinjaMutex\UnrecoverableMutexException;
-
 /**
  * Abstract lock implementor
  *
@@ -42,30 +40,6 @@ public function __clone()
         $this->locks = array();
     }
 
-    /**
-     * Try to release any obtained locks when object is destroyed
-     *
-     * This is a safe guard for cases when your php script dies unexpectedly.
-     * It's not guaranteed it will work either.
-     *
-     * You should not depend on __destruct() to release your locks,
-     * instead release them with `$released = $this->releaseLock()`A
-     * and check `$released` if lock was properly released
-     * @throws UnrecoverableMutexException
-     */
-    public function __destruct()
-    {
-        foreach ($this->locks as $name => $v) {
-            $released = $this->releaseLock($name);
-            if (!$released) {
-                throw new UnrecoverableMutexException(sprintf(
-                    'Cannot release lock in __destruct(): %s',
-                    $name
-                ));
-            }
-        }
-    }
-
     /**
      * Acquire lock
      *

src/Lock/MySQLPDOLock.php

@@ -178,13 +178,4 @@ protected function setupPDO($name)
 
         return true;
     }
-
-    public function __destruct()
-    {
-        parent::__destruct();
-
-        foreach($this->pdo as $name => $pdo) {
-            unset($this->pdo[$name]);
-        }
-    }
 }

src/Mutex.php

@@ -82,30 +82,6 @@ public function releaseLock()
         return false;
     }
 
-    /**
-     * Try to release any obtained locks when object is destroyed
-     *
-     * This is a safe guard for cases when your php script dies unexpectedly.
-     * It's not guaranteed it will work either.
-     *
-     * You should not depend on __destruct() to release your locks,
-     * instead release them with `$released = $this->releaseLock()`A
-     * and check `$released` if lock was properly released
-     * @throws UnrecoverableMutexException
-     */
-    public function __destruct()
-    {
-        while ($this->isAcquired()) {
-            $released = $this->releaseLock();
-            if (!$released) {
-                throw new UnrecoverableMutexException(sprintf(
-                    'Cannot release lock in Mutex __destruct(): %s',
-                    $this->name
-                ));
-            }
-        }
-    }
-
     /**
      * Check if Mutex is acquired
      *

tests/Lock/LockTest.php

@@ -14,7 +14,6 @@
 use NinjaMutex\Tests\AbstractTest;
 use NinjaMutex\Tests\Lock\Fabric\LockFabricWithExpirationInterface;
 use NinjaMutex\Tests\Mock\PermanentServiceInterface;
-use NinjaMutex\UnrecoverableMutexException;
 
 /**
  * Tests for Locks
@@ -128,43 +127,6 @@ public function testIfLockIsReleasedAfterLockImplementorIsDestroyed(LockInterfac
         $lockImplementor->releaseLock($name);
     }
 
-    /**
-     * @issue https://github.com/arvenil/ninja-mutex/pull/4
-     * It's not working for hhvm, see below link to understand limitation
-     * https://github.com/facebook/hhvm/blob/af329776c9f740cc1c8c4791f673ba5aa49042ce/hphp/doc/inconsistencies#L40-L45
-     *
-     * @dataProvider lockImplementorWithBackendProvider
-     * @param LockInterface $lockImplementor
-     * @param PermanentServiceInterface $backend
-     */
-    public function testIfLockDestructorThrowsWhenBackendIsUnavailable(LockInterface $lockImplementor, PermanentServiceInterface $backend)
-    {
-        $name = "forfiter";
-
-        $this->assertFalse($lockImplementor->isLocked($name));
-        $this->assertTrue($lockImplementor->acquireLock($name, 0));
-        $this->assertTrue($lockImplementor->isLocked($name));
-
-        // make backend unavailable
-        $backend->setAvailable(false);
-
-        try {
-            // explicit __destructor() call, should throw UnrecoverableMutexException
-            $lockImplementor->__destruct();
-        } catch (UnrecoverableMutexException $e) {
-            // make backend available again
-            $backend->setAvailable(true);
-            // release lock
-            $this->assertTrue($lockImplementor->releaseLock($name));
-            $this->assertFalse($lockImplementor->releaseLock($name));
-            $this->assertFalse($lockImplementor->isLocked($name));
-
-            return;
-        }
-
-        $this->fail('An expected exception has not been raised.');
-    }
-
     /**
      * @issue https://github.com/arvenil/ninja-mutex/issues/12
      * @medium Timeout for test increased to ~5s http://stackoverflow.com/a/10535787/916440

tests/Mock/MockLock.php

@@ -87,8 +87,4 @@ public function setAvailable($available)
     {
         $this->available = (bool)$available;
     }
-
-    public function __destruct()
-    {
-    }
 }

tests/Mock/MockPDO.php

@@ -150,12 +150,4 @@ public function quote($string, $type = PDO::PARAM_STR)
     {
         return $string;
     }
-
-    public function __destruct()
-    {
-        foreach ($this->current as $k => $v) {
-            unset(self::$data[$k]);
-            unset($this->current[$k]);
-        }
-    }
 }