Merge pull request #40 from Segaja/fix_let_statements

start to fix let statements to make vimp compatible with FF 35.0a1

Author: maxauthority

common/content/commandline.js

@@ -1464,9 +1464,9 @@ const CommandLine = Module("commandline", {
             commands.add([command.name],
                 command.description,
                 function (args) {
-                    var str = args.literalArg;
+                    let str = args.literalArg;
 
-                    let str = CommandLine.echoArgumentToString(str, true);
+                    str = CommandLine.echoArgumentToString(str, true);
                     if (str != null)
                         command.action(str);
                 }, {

common/content/commands.js

@@ -548,6 +548,8 @@ const Commands = Module("commands", {
         var onlyArgumentsRemaining = allowUnknownOptions || (options.length == 0 && subCommands.length == 0) || false; // after a -- has been found
         var arg = null;
         var count = 0; // the length of the argument
+        var quote = null;
+        var error = null;
         var i = 0;
         var completeOpts;
 
@@ -692,7 +694,7 @@ const Commands = Module("commands", {
             }
 
             // if not an option, treat this token as an argument
-            let [count, arg, quote, error] = getNextArg(sub);
+            [count, arg, quote, error] = getNextArg(sub);
             liberator.assert(!error, error);
 
             if (complete) {

common/content/liberator.js

@@ -1717,9 +1717,9 @@ const Liberator = Module("liberator", {
                     return void liberator.echoerr("No usage information for: " + args[0]);
 
                 if (args[0])
-                    var usage = template.genericOutput(config.name + " Usage", usage[args[0]]());
+                    usage = template.genericOutput(config.name + " Usage", usage[args[0]]());
                 else
-                    var usage = template.genericOutput(config.name + " Usage", xml`${ usage["mappings"]() }<br/>${ usage["commands"]() }<br/>${ usage["options"]()}`);
+                    usage = template.genericOutput(config.name + " Usage", xml`${ usage["mappings"]() }<br/>${ usage["commands"]() }<br/>${ usage["options"]()}`);
                 liberator.echo(usage, commandline.FORCE_MULTILINE);
             }, {
                 argCount: "?",

common/content/options.js

@@ -123,11 +123,11 @@ const Option = Class("Option", {
         else
             scope = this.scope;
 
+        let value;
         // Options with a custom getter are always responsible for returning a meaningful value
         if (this.getter)
             return liberator.trapErrors(this.getter, this, value);
 
-        let value;
         if (liberator.has("tabs") && (scope & Option.SCOPE_LOCAL))
             value = tabs.options[this.name];
         if ((scope & Option.SCOPE_GLOBAL) && (value == undefined))