Rename NoMutex class to NullMutex (#74)

Author: mvorisek

README.md

@@ -173,7 +173,7 @@ implementations or create/extend your own implementation.
 #### FlockMutex
 
 The **FlockMutex** is a lock implementation based on
-[`flock()`](http://php.net/manual/en/function.flock.php).
+[`flock()`](https://php.net/manual/en/function.flock.php).
 
 Example:
 ```php
@@ -194,7 +194,7 @@ extension if possible or busy waiting if not.
 #### MemcachedMutex
 
 The **MemcachedMutex** is a spinlock implementation which uses the
-[`Memcached` extension](http://php.net/manual/en/book.memcached.php).
+[`Memcached` extension](https://php.net/manual/en/book.memcached.php).
 
 Example:
 ```php
@@ -215,7 +215,7 @@ $mutex->synchronized(function () use ($bankAccount, $amount) {
 #### RedisMutex
 
 The **RedisMutex** is the distributed lock implementation of
-[RedLock](http://redis.io/topics/distlock) which supports the
+[RedLock](https://redis.io/topics/distlock#the-redlock-algorithm) which supports the
 [`phpredis` extension](https://github.com/phpredis/phpredis)
 or [`Predis` API](https://github.com/nrk/predis).
 
@@ -244,7 +244,7 @@ $mutex->synchronized(function () use ($bankAccount, $amount) {
 #### SemaphoreMutex
 
 The **SemaphoreMutex** is a lock implementation based on
-[Semaphore](http://php.net/manual/en/ref.sem.php).
+[Semaphore](https://php.net/manual/en/ref.sem.php).
 
 Example:
 ```php
@@ -330,9 +330,9 @@ Commercial support is available.
 
 This project is free and is licensed under the MIT.
 
-[1]: http://semver.org/
+[1]: https://semver.org/
 [2]: https://github.com/nrk/predis
-[3]: http://php.net/manual/en/book.pcntl.php
+[3]: https://php.net/manual/en/book.pcntl.php
 [4]: https://getcomposer.org/
 [5]: https://github.com/php-lock/lock/blob/3ca295ccda/src/Mutex/Mutex.php#L15
 [6]: https://github.com/php-lock/lock/blob/3ca295ccda/src/Mutex/Mutex.php#L38

src/Exception/ExecutionOutsideLockException.php

@@ -9,8 +9,9 @@
 
 /**
  * This exception should be thrown when for example the lock is released or the
- * lock times out before the critical code has finished execution. This is a
- * serious exception. Side effects might have happened while the critical code
+ * lock times out before the critical code has finished execution.
+ *
+ * This is a serious exception. Side effects might have happened while the critical code
  * was executed outside of the lock which should not be trusted to be valid.
  *
  * Should only be used in contexts where the lock is being released.

src/Mutex/AbstractRedlockMutex.php

@@ -16,7 +16,9 @@
  *
  * @template TClient of object
  *
- * @see http://redis.io/topics/distlock
+ * @internal
+ *
+ * @see https://redis.io/topics/distlock#the-redlock-algorithm
  */
 abstract class AbstractRedlockMutex extends AbstractSpinlockWithTokenMutex implements LoggerAwareInterface
 {

src/Mutex/Mutex.php

@@ -38,8 +38,7 @@ public function synchronized(callable $code);
     /**
      * Performs a double-checked locking pattern.
      *
-     * Call {@link \Malkusch\Lock\Util\DoubleCheckedLocking::then()} on the
-     * returned object.
+     * Call {@link DoubleCheckedLocking::then()} on the returned object.
      *
      * Example:
      * <code>

src/Mutex/MySQLMutex.php

@@ -27,7 +27,7 @@ public function __construct(\PDO $PDO, string $name, float $acquireTimeout = 0)
         $namePrefix = LockUtil::getInstance()->getKeyPrefix() . ':';
 
         if (\strlen($name) > 64 - \strlen($namePrefix)) {
-            throw new \InvalidArgumentException('The maximum length of the lock name is ' . (64 - \strlen($namePrefix)) . ' characters');
+            throw new \InvalidArgumentException('The maximum length of the database lock name is ' . (64 - \strlen($namePrefix)) . ' characters');
         }
 
         $this->name = $namePrefix . $name;

src/Mutex/NoMutex.php renamed to src/Mutex/NullMutex.php

@@ -5,12 +5,9 @@
 namespace Malkusch\Lock\Mutex;
 
 /**
- * This mutex doesn't lock at all.
- *
- * Synchronization is not provided! This mutex is just implementing the
- * interface without locking.
+ * This mutex does not lock at all.
  */
-class NoMutex extends AbstractMutex
+class NullMutex extends AbstractMutex
 {
     #[\Override]
     public function synchronized(callable $code)

src/Mutex/RedisMutex.php

@@ -17,7 +17,7 @@
  *
  * @extends AbstractRedlockMutex<TClient>
  *
- * @see http://redis.io/topics/distlock
+ * @see https://redis.io/topics/distlock#the-redlock-algorithm
  */
 class RedisMutex extends AbstractRedlockMutex
 {
@@ -122,7 +122,7 @@ protected function evalScript(object $client, string $luaScript, array $keys, ar
             }
         } else {
             try {
-                return $client->eval($luaScript, count($keys), ...[...$keys, ...$arguments]);
+                return $client->eval($luaScript, count($keys), ...$keys, ...$arguments);
             } catch (PredisException $e) {
                 throw new LockReleaseException('Failed to release lock', 0, $e);
             }

src/Mutex/SemaphoreMutex.php

@@ -12,19 +12,19 @@
  */
 class SemaphoreMutex extends AbstractLockMutex
 {
-    /** @var \SysvSemaphore|resource The semaphore id */
+    /** @var \SysvSemaphore|resource */
     private $semaphore;
 
     /**
-     * Use {@link sem_get()} to create the semaphore id.
+     * Use {@link sem_get()} to create the semaphore.
      *
      * Example:
      * <code>
      * $semaphore = sem_get(ftok(__FILE__, 'a'));
      * $mutex = new SemaphoreMutex($semaphore);
      * </code>
      *
-     * @param \SysvSemaphore|resource $semaphore The semaphore id
+     * @param \SysvSemaphore|resource $semaphore
      */
     public function __construct($semaphore)
     {

src/Util/DoubleCheckedLocking.php

@@ -12,7 +12,7 @@
  * The double-checked locking pattern.
  *
  * You should not instantiate this class directly. Use
- * {@link \Malkusch\Lock\Mutex\Mutex::check()}.
+ * {@link Mutex::check()}.
  *
  * @internal
  */
@@ -24,9 +24,8 @@ class DoubleCheckedLocking
     private $check;
 
     /**
-     * @param Mutex            $mutex Provides methods for exclusive code execution
-     * @param callable(): bool $check Callback that decides if the lock should be acquired and if the critical code
-     *                                callback should be executed after acquiring the lock
+     * @param callable(): bool $check Callback that decides if the lock should be acquired and is rechecked
+     *                                after a lock has been acquired
      */
     public function __construct(Mutex $mutex, callable $check)
     {
@@ -35,17 +34,12 @@ public function __construct(Mutex $mutex, callable $check)
     }
 
     /**
-     * Executes a synchronized callback only after the check callback passes
-     * before and after acquiring the lock.
-     *
-     * If then returns boolean boolean false, the check did not pass before or
-     * after acquiring the lock. A boolean false can also be returned from the
-     * critical code callback to indicate that processing did not occure or has
-     * failed. It is up to the user to decide the last point.
+     * Executes a block of code only after the check callback passes
+     * before and after acquiring a lock.
      *
      * @template T
      *
-     * @param callable(): T $code The critical code callback
+     * @param callable(): T $code
      *
      * @return T|false False if check did not pass
      *

src/Util/Loop.php

@@ -23,7 +23,7 @@ class Loop
     private bool $looping = false;
 
     /**
-     * Notifies that this was the last iteration.
+     * Notify that this was the last iteration.
      */
     public function end(): void
     {
@@ -35,7 +35,7 @@ public function end(): void
      *
      * The code has to be designed in a way that it can be repeated without any
      * side effects. When execution was successful it should notify that event
-     * by calling {@link \Malkusch\Lock\Util\Loop::end()}. I.e. the only side
+     * by calling {@link Loop::end()}. I.e. the only side
      * effects of the code may happen after a successful execution.
      *
      * If the code throws an exception it will stop repeating the execution.