Enforce sprintf for exceptions

Author: dantleech

Options.php

@@ -225,7 +225,7 @@ public function get($option)
         $this->reading = true;
 
         if (!array_key_exists($option, $this->options)) {
-            throw new \OutOfBoundsException('The option "'.$option.'" does not exist.');
+            throw new \OutOfBoundsException(sprintf('The option "%s" does not exist.', $option));
         }
 
         if (isset($this->lazy[$option])) {
@@ -459,7 +459,7 @@ private function resolve($option)
                 }
             }
 
-            throw new OptionDefinitionException('The options "'.implode('", "', $conflicts).'" have a cyclic dependency.');
+            throw new OptionDefinitionException(sprintf('The options "%s" have a cyclic dependency.', implode('", "', $conflicts)));
         }
 
         $this->lock[$option] = true;
@@ -497,7 +497,7 @@ private function normalize($option)
                 }
             }
 
-            throw new OptionDefinitionException('The options "'.implode('", "', $conflicts).'" have a cyclic dependency.');
+            throw new OptionDefinitionException(sprintf('The options "%s" have a cyclic dependency.', implode('", "', $conflicts)));
         }
 
         /** @var \Closure $normalizer */