Code style updates.

Author: bobthecow

src/Context.php

@@ -40,10 +40,10 @@
  */
 class Context implements \ArrayAccess
 {
-    private array $keys   = array();
-    private array $values = array();
-    private array $frozen = array();
-    private array $raw    = array();
+    private array $keys = [];
+    private array $values = [];
+    private array $frozen = [];
+    private array $raw = [];
 
     private $shared;
     private $protected;
@@ -56,10 +56,10 @@ class Context implements \ArrayAccess
      *
      * @param array $values (default: array())
      */
-    public function __construct(array $values = array())
+    public function __construct(array $values = [])
     {
-        $this->shared    = new \SplObjectStorage;
-        $this->protected = new \SplObjectStorage;
+        $this->shared = new \SplObjectStorage();
+        $this->protected = new \SplObjectStorage();
 
         foreach ($values as $key => $value) {
             $this->offsetSet($key, $value);
@@ -71,7 +71,7 @@ public function __construct(array $values = array())
      *
      * @param string $name The unique name for the fact
      *
-     * @return boolean
+     * @return bool
      */
     public function offsetExists($name): bool
     {
@@ -83,9 +83,9 @@ public function offsetExists($name): bool
      *
      * @param string $name The unique name for the fact
      *
-     * @return mixed The resolved value of the fact
-     *
      * @throws \InvalidArgumentException if the name is not defined
+     *
+     * @return mixed The resolved value of the fact
      */
     #[\ReturnTypeWillChange]
     public function offsetGet($name)
@@ -104,7 +104,7 @@ public function offsetGet($name)
         // If this is a shared value, resolve, freeze, and return the result
         if ($this->shared->contains($value)) {
             $this->frozen[$name] = true;
-            $this->raw[$name]    = $value;
+            $this->raw[$name] = $value;
 
             return $this->values[$name] = $value($this);
         }
@@ -130,12 +130,12 @@ public function offsetSet($name, $value): void
             throw new \RuntimeException(sprintf('Cannot override frozen fact "%s".', $name));
         }
 
-        $this->keys[$name]   = true;
+        $this->keys[$name] = true;
         $this->values[$name] = $value;
     }
 
     /**
-     * Unset a fact
+     * Unset a fact.
      *
      * @param string $name The unique name for the fact
      */
@@ -159,9 +159,9 @@ public function offsetUnset($name): void
      *
      * @param callable $callable A fact callable to share
      *
-     * @return callable The passed callable
-     *
      * @throws \InvalidArgumentException if the callable is not a Closure or invokable object
+     *
+     * @return callable The passed callable
      */
     public function share($callable)
     {
@@ -182,9 +182,9 @@ public function share($callable)
      *
      * @param callable $callable A callable to protect from being evaluated
      *
-     * @return callable The passed callable
-     *
      * @throws \InvalidArgumentException if the callable is not a Closure or invokable object
+     *
+     * @return callable The passed callable
      */
     public function protect($callable)
     {
@@ -202,9 +202,9 @@ public function protect($callable)
      *
      * @param string $name The unique name for the fact
      *
-     * @return mixed The value of the fact or the closure defining the fact
-     *
      * @throws \InvalidArgumentException if the name is not defined
+     *
+     * @return mixed The value of the fact or the closure defining the fact
      */
     public function raw($name)
     {
@@ -234,7 +234,7 @@ public function keys()
      *
      * @param mixed $callable
      *
-     * @return boolean
+     * @return bool
      */
     protected function isCallable($callable)
     {

src/Operator.php

@@ -20,7 +20,7 @@ abstract class Operator
     public const BINARY = 'BINARY';
     public const MULTIPLE = 'MULTIPLE';
 
-    protected $operands = array();
+    protected $operands = [];
 
     /**
      * @param array $operands
@@ -37,17 +37,17 @@ public function getOperands()
         switch ($this->getOperandCardinality()) {
             case self::UNARY:
                 if (1 != count($this->operands)) {
-                    throw new \LogicException(get_class($this) . ' takes only 1 operand');
+                    throw new \LogicException(get_class($this).' takes only 1 operand');
                 }
                 break;
             case self::BINARY:
                 if (2 != count($this->operands)) {
-                    throw new \LogicException(get_class($this) . ' takes 2 operands');
+                    throw new \LogicException(get_class($this).' takes 2 operands');
                 }
                 break;
             case self::MULTIPLE:
                 if (0 == count($this->operands)) {
-                    throw new \LogicException(get_class($this) . ' takes at least 1 operand');
+                    throw new \LogicException(get_class($this).' takes at least 1 operand');
                 }
                 break;
         }
@@ -56,5 +56,6 @@ public function getOperands()
     }
 
     abstract public function addOperand($operand);
+
     abstract protected function getOperandCardinality();
 }

src/Operator/Addition.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * An Addition Arithmetic Operator
+ * An Addition Arithmetic Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Ceil.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Ceil Math Operator
+ * A Ceil Math Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Complement.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Complement Set Operator
+ * A Complement Set Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/ContainsSubset.php

@@ -25,7 +25,7 @@ class ContainsSubset extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/Division.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Division Arithmetic Operator
+ * A Division Arithmetic Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/DoesNotContainSubset.php

@@ -25,7 +25,7 @@ class DoesNotContainSubset extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/EndsWith.php

@@ -25,7 +25,7 @@ class EndsWith extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/EndsWithInsensitive.php

@@ -25,7 +25,7 @@ class EndsWithInsensitive extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/EqualTo.php

@@ -25,7 +25,7 @@ class EqualTo extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/Exponentiate.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * An Exponentiate Math Operator
+ * An Exponentiate Math Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Floor.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Floor Math Operator
+ * A Floor Math Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/GreaterThan.php

@@ -25,7 +25,7 @@ class GreaterThan extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/GreaterThanOrEqualTo.php

@@ -25,7 +25,7 @@ class GreaterThanOrEqualTo extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/Intersect.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Set Intersection Operator
+ * A Set Intersection Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/LessThan.php

@@ -25,7 +25,7 @@ class LessThan extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/LessThanOrEqualTo.php

@@ -25,7 +25,7 @@ class LessThanOrEqualTo extends VariableOperator implements Proposition
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/LogicalAnd.php

@@ -24,7 +24,7 @@ class LogicalAnd extends LogicalOperator
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/LogicalNot.php

@@ -24,7 +24,7 @@ class LogicalNot extends LogicalOperator
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/LogicalOperator.php

@@ -14,18 +14,18 @@
 use Ruler\Proposition;
 
 /**
- * Logical operator base class
+ * Logical operator base class.
  *
  * @author Justin Hileman <[email protected]>
  */
 abstract class LogicalOperator extends PropositionOperator implements Proposition
 {
     /**
-     * array of propositions
+     * array of propositions.
      *
      * @param array $props
      */
-    public function __construct(array $props = array())
+    public function __construct(array $props = [])
     {
         foreach ($props as $operand) {
             $this->addOperand($operand);

src/Operator/LogicalOr.php

@@ -24,7 +24,7 @@ class LogicalOr extends LogicalOperator
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/LogicalXor.php

@@ -24,7 +24,7 @@ class LogicalXor extends LogicalOperator
     /**
      * @param Context $context Context with which to evaluate this Proposition
      *
-     * @return boolean
+     * @return bool
      */
     public function evaluate(Context $context)
     {

src/Operator/Max.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A set max operator
+ * A set max operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Min.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A set min operator
+ * A set min operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Modulo.php

@@ -16,7 +16,7 @@
 use Ruler\VariableOperand;
 
 /**
- * A Modulo Arithmetic Operator
+ * A Modulo Arithmetic Operator.
  *
  * @author Jordan Raub <[email protected]>
  */

src/Operator/Multiplication.php

@@ -12,11 +12,11 @@
 namespace Ruler\Operator;
 
 use Ruler\Context;
-use Ruler\VariableOperand;
 use Ruler\Value;
+use Ruler\VariableOperand;
 
 /**
- * A Multiplication Arithmetic Operator
+ * A Multiplication Arithmetic Operator.
  *
  * @author Jordan Raub <[email protected]>
  */