Use null as defaults

src/Worker/DefaultPool.php

@@ -55,14 +55,16 @@ class DefaultPool implements Pool
     /**
      * Creates a new worker pool.
      *
-     * @param int $minSize The minimum number of workers the pool should spawn. Defaults to `Pool::DEFAULT_MIN_SIZE`.
-     * @param int $maxSize The maximum number of workers the pool should spawn. Defaults to `Pool::DEFAULT_MAX_SIZE`.
+     * @param int|null $minSize The minimum number of workers the pool should spawn.
+     *     Defaults to `Pool::DEFAULT_MIN_SIZE`.
+     * @param int|null $maxSize The maximum number of workers the pool should spawn.
+     *     Defaults to `Pool::DEFAULT_MAX_SIZE`.
      * @param \Icicle\Concurrent\Worker\WorkerFactory|null $factory A worker factory to be used to create
      *     new workers.
      *
      * @throws \Icicle\Exception\InvalidArgumentError
      */
-    public function __construct($minSize = 0, $maxSize = 0, WorkerFactory $factory = null)
+    public function __construct($minSize = null, $maxSize = null, WorkerFactory $factory = null)
     {
         $minSize = $minSize ?: self::DEFAULT_MIN_SIZE;
         $maxSize = $maxSize ?: self::DEFAULT_MAX_SIZE;

src/Worker/DefaultQueue.php

@@ -39,13 +39,15 @@ class DefaultQueue implements Queue
     private $running = false;
 
     /**
-     * @param int $minSize
-     * @param int $maxSize
+     * @param int|null $minSize The minimum number of workers the queue should spawn.
+     *     Defaults to `Queue::DEFAULT_MIN_SIZE`.
+     * @param int|null $maxSize The maximum number of workers the queue should spawn.
+     *     Defaults to `Queue::DEFAULT_MAX_SIZE`.
      * @param \Icicle\Concurrent\Worker\WorkerFactory|null $factory Factory used to create new workers.
      *
      * @throws \Icicle\Exception\InvalidArgumentError If the min or max size are invalid.
      */
-    public function __construct($minSize = 0, $maxSize = 0, WorkerFactory $factory = null)
+    public function __construct($minSize = null, $maxSize = null, WorkerFactory $factory = null)
     {
         $minSize = $minSize ?: self::DEFAULT_MIN_SIZE;
         $maxSize = $maxSize ?: self::DEFAULT_MAX_SIZE;

tests/Worker/AbstractPoolTest.php

@@ -14,7 +14,7 @@ abstract class AbstractPoolTest extends TestCase
      *
      * @return \Icicle\Concurrent\Worker\Pool
      */
-    abstract protected function createPool($min = 0, $max = 0);
+    abstract protected function createPool($min = null, $max = null);
 
     public function testIsRunning()
     {

tests/Worker/AbstractQueueTest.php

@@ -15,7 +15,7 @@ abstract class AbstractQueueTest extends TestCase
      *
      * @return \Icicle\Concurrent\Worker\Queue
      */
-    abstract protected function createQueue($min = 0, $max = 0);
+    abstract protected function createQueue($min = null, $max = null);
 
     public function testIsRunning()
     {

tests/Worker/ForkPoolTest.php

@@ -11,7 +11,7 @@ use Icicle\Concurrent\Worker\WorkerFork;
  */
 class ForkPoolTest extends AbstractPoolTest
 {
-    protected function createPool($min = 0, $max = 0)
+    protected function createPool($min = null, $max = null)
     {
         $factory = $this->getMock(WorkerFactory::class);
         $factory->method('create')->will($this->returnCallback(function () {

tests/Worker/ForkQueueTest.php

@@ -11,7 +11,7 @@ use Icicle\Concurrent\Worker\WorkerFork;
  */
 class ForkQueueTest extends AbstractQueueTest
 {
-    protected function createQueue($min = 0, $max = 0)
+    protected function createQueue($min = null, $max = null)
     {
         $factory = $this->getMock(WorkerFactory::class);
         $factory->method('create')->will($this->returnCallback(function () {

tests/Worker/ProcessPoolTest.php

@@ -10,7 +10,7 @@ use Icicle\Concurrent\Worker\WorkerProcess;
  */
 class ProcessPoolTest extends AbstractPoolTest
 {
-    protected function createPool($min = 0, $max = 0)
+    protected function createPool($min = null, $max = null)
     {
         $factory = $this->getMock(WorkerFactory::class);
         $factory->method('create')->will($this->returnCallback(function () {

tests/Worker/ThreadPoolTest.php

@@ -11,7 +11,7 @@ use Icicle\Concurrent\Worker\WorkerThread;
  */
 class ThreadPoolTest extends AbstractPoolTest
 {
-    protected function createPool($min = 0, $max = 0)
+    protected function createPool($min = null, $max = null)
     {
         $factory = $this->getMock(WorkerFactory::class);
         $factory->method('create')->will($this->returnCallback(function () {

tests/Worker/ThreadQueueTest.php

@@ -11,7 +11,7 @@ use Icicle\Concurrent\Worker\WorkerThread;
  */
 class ThreadQueueTest extends AbstractQueueTest
 {
-    protected function createQueue($min = 0, $max = 0)
+    protected function createQueue($min = null, $max = null)
     {
         $factory = $this->getMock(WorkerFactory::class);
         $factory->method('create')->will($this->returnCallback(function () {