Move next ID initialization

amphp · Aug 3, 2024 · de3d8d4 · de3d8d4
1 parent 25c7662
commit de3d8d4
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/PosixSemaphore.php b/src/PosixSemaphore.php
@@ -40,10 +40,6 @@ public static function create(int $maxLocks, int $permissions = 0600): self
             throw new \Error('Number of locks must be greater than 0, got ' . $maxLocks);
         }
 
-        if (self::$nextId === 0) {
-            self::$nextId = \random_int(1, self::MAX_ID);
-        }
-
         \set_error_handler(static function (int $errno, string $errstr): bool {
             if (!\str_contains($errstr, 'No space left on device') && \str_contains($errstr, 'Failed for key')) {
                 return true;
@@ -77,6 +73,10 @@ public static function create(int $maxLocks, int $permissions = 0600): self
 
     private static function getNextId(): int
     {
+        if (self::$nextId === 0) {
+            return self::$nextId = \random_int(1, self::MAX_ID);
+        }
+
         return self::$nextId = self::$nextId % self::MAX_ID + 1;
     }