Merge pull request #514 from lkolbly/proxyFix

Changed let scope, added undefined check.
This commit is contained in:
Romain Beaumont 2017-08-11 00:11:24 +02:00 committed by GitHub
commit c01497816b

View File

@ -41,7 +41,8 @@ let printAllNames = false;
const printNameWhitelist = {}; const printNameWhitelist = {};
const printNameBlacklist = {}; const printNameBlacklist = {};
(function() { (function() {
for(let i = 0; i < args.length; i++) { let i = 0;
for(i = 0; i < args.length; i++) {
const option = args[i]; const option = args[i];
if(!/^-/.test(option)) break; if(!/^-/.test(option)) break;
if(option === "--dump-all") { if(option === "--dump-all") {
@ -180,6 +181,6 @@ function shouldDump(name, direction) {
return matches(printNameWhitelist[name]); return matches(printNameWhitelist[name]);
function matches(result) { function matches(result) {
return result !== null && result.indexOf(direction) !== -1; return result !== undefined && result !== null && result.indexOf(direction) !== -1;
} }
} }