fix: the getBinaryPath function only need to execute once enough (#…

…2620) * fix: the function only need to execute once enough * fix: typo `;`
sass · Nov 9, 2019 · 7293699 · 7293699
1 parent ac145b1
commit 7293699
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/binding.js b/lib/binding.js
@@ -8,13 +8,15 @@ var errors = require('./errors');
  * Require binding
  */
 module.exports = function(ext) {
-  if (!ext.hasBinary(ext.getBinaryPath())) {
+  var binaryPath = ext.getBinaryPath();
+
+  if (!ext.hasBinary(binaryPath)) {
     if (!ext.isSupportedEnvironment()) {
       throw new Error(errors.unsupportedEnvironment());
     } else {
       throw new Error(errors.missingBinary());
     }
   }
 
-  return require(ext.getBinaryPath());
+  return require(binaryPath);
 };