Dotenv: Unused Loader instance in __construct

jrchamp · web-flow · commit ade362775ca9 · 2016-11-18T15:00:19.000-05:00
loadData() will always create a Loader instance, so creating one in Dotenv::__construct() is pointless.
diff --git a/src/Dotenv.php b/src/Dotenv.php
@@ -35,7 +35,6 @@ class Dotenv
     public function __construct($path, $file = '.env')
     {
         $this->filePath = $this->getFilePath($path, $file);
-        $this->loader = new Loader($this->filePath, true);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,6 @@ class Dotenv`
`35`	`35`	`public function __construct($path, $file = '.env')`
`36`	`36`	`{`
`37`	`37`	`$this->filePath = $this->getFilePath($path, $file);`
`38`		`- $this->loader = new Loader($this->filePath, true);`
`39`	`38`	`}`
`40`	`39`
`41`	`40`	`/**`