Eliminate unnecessary else branching

Author: geekwright

src/PoFile.php

@@ -165,7 +165,7 @@ public function addEntry(PoEntry $entry, bool $replace = true): bool
     {
         $key = $this->createKeyFromEntry($entry);
 
-        // some entires, such as obsolete entries, have no key
+        // some entries, such as obsolete entries, have no key
         // for some uses, these are dead weight - need better strategy for that case
         if (empty($key)) {
             $this->unkeyedEntries[] = $entry;
@@ -174,10 +174,10 @@ public function addEntry(PoEntry $entry, bool $replace = true): bool
 
         if (isset($this->entries[$key]) && !$replace) {
             return false;
-        } else {
-            $this->entries[$key] = $entry;
-            return true;
         }
+
+        $this->entries[$key] = $entry;
+        return true;
     }
 
     /**

src/PoHeader.php

@@ -105,12 +105,12 @@ public function setHeader(string $key, string $value): void
     {
         $this->buildStructuredHeaders();
         $lkey = strtolower($key);
+        $newHeader = array('key' => $key, 'value' => $value);
         if (isset($this->structuredHeaders[$lkey])) {
-            $this->structuredHeaders[$lkey]['value'] = $value;
-        } else {
-            $newHeader = array('key' => $key, 'value' => $value);
-            $this->structuredHeaders[$lkey] = $newHeader;
+            $newHeader = $this->structuredHeaders[$lkey];
+            $newHeader['value'] = $value;
         }
+        $this->structuredHeaders[$lkey] = $newHeader;
         $this->storeStructuredHeader();
     }
 

src/PoInitAbstract.php

@@ -172,9 +172,8 @@ public function addPgettextTags($tags): void
      */
     public function msginitFile(string $filename): PoFile
     {
-        if (!is_readable($filename)) {
-            $source = false;
-        } else {
+        $source = false;
+        if (is_readable($filename)) {
             $source = file_get_contents($filename);
         }
         if (false===$source) {