Remove guard that specifically maps "BYTE"

chrisullyott · chrisullyott · commit 314b6b8d94db · 2019-06-06T21:48:07.000-07:00
This just adds complexity and doesn't mean much for performance.
diff --git a/src/FileSize/UnitMap/UnitMapper.php b/src/FileSize/UnitMap/UnitMapper.php
@@ -46,10 +46,6 @@ public function __construct()
      */
     public function keyFromString($unitString)
     {
-        if ($unitString === UnitMap::BYTE) {
-            return UnitMap::BYTE;
-        }
-
         if (isset($this->cache[$unitString])) {
             return $this->cache[$unitString];
         }

Original file line number	Diff line number	Diff line change
`@@ -46,10 +46,6 @@ public function __construct()`
`46`	`46`	`*/`
`47`	`47`	`public function keyFromString($unitString)`
`48`	`48`	`{`
`49`		`- if ($unitString === UnitMap::BYTE) {`
`50`		`- return UnitMap::BYTE;`
`51`		`- }`
`52`		`-`
`53`	`49`	`if (isset($this->cache[$unitString])) {`
`54`	`50`	`return $this->cache[$unitString];`
`55`	`51`	`}`