Move the parser instantiation below the error checks as it is not used before it. Would be extra resources if the thing is thrown. Also more the $this->path setting into else since we don't want the whole string to be set as a path if we are passing in a large chunk of data.

Author: helgi

src/Pyrus/PackageFile.php

@@ -34,11 +34,10 @@ function __construct($package, $class = 'Pyrus\PackageFile\v2', $isString = fals
             return $this->info = $package;
         }
 
-        $this->path = $package;
-        $parser = new PackageFile\Parser\v2;
         if ($isString) {
             $data = $package;
         } else {
+            $this->path = $package;
             $data = file_get_contents($package);
         }
 
@@ -47,6 +46,7 @@ function __construct($package, $class = 'Pyrus\PackageFile\v2', $isString = fals
                 . $package . ' or file was empty.');
         }
 
+        $parser = new PackageFile\Parser\v2;
         $this->info = $parser->parse($data, $package, $class);
     }