Remove unecessery semicolons (#515)

vimperator · Jun 8, 2016 · 13056b5 · 13056b5
1 parent a2e4a4f
commit 13056b5
Show file tree

Hide file tree

Showing 16 changed files with 36 additions and 36 deletions.
diff --git a/common/components/protocols.js b/common/components/protocols.js
@@ -30,7 +30,7 @@ const systemPrincipal = channel.owner;
 channel.cancel(NS_BINDING_ABORTED);
 delete channel;
 
-function dataURL(type, data) "data:" + (type || "application/xml;encoding=UTF-8") + "," + encodeURIComponent(data);
+function dataURL(type, data) "data:" + (type || "application/xml;encoding=UTF-8") + "," + encodeURIComponent(data)
 function makeChannel(url, orig) {
     if (typeof url == "function")
         url = dataURL.apply(null, url());
@@ -40,7 +40,7 @@ function makeChannel(url, orig) {
     channel.originalURI = orig;
     return channel;
 }
-function fakeChannel(orig) makeChannel("chrome://liberator/content/does/not/exist", orig);
+function fakeChannel(orig) makeChannel("chrome://liberator/content/does/not/exist", orig)
 function redirect(to, orig) {
     //xxx: escape
     let html = '<html><head><meta http-equiv="Refresh" content="' + ("0;" + to).replace(/"/g, "&quot;") + '"/></head></html>';
@@ -161,6 +161,6 @@ var components = [ChromeData, Liberator];
 if(XPCOMUtils.generateNSGetFactory)
     var NSGetFactory = XPCOMUtils.generateNSGetFactory(components);
 else
-    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule(components);
+    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule(components)
 
 // vim: set fdm=marker sw=4 ts=4 et:
diff --git a/common/content/base.js b/common/content/base.js
@@ -184,7 +184,7 @@ function curry(fn, length, self, acc) {
         return fn;
 
     // Close over function with 'this'
-    function close(self, fn) function () fn.apply(self, Array.slice(arguments));
+    function close(self, fn) function () fn.apply(self, Array.slice(arguments))
 
     if (acc == null)
         acc = [];
@@ -366,7 +366,7 @@ Class.prototype = {
      */
     setTimeout: function (callback, timeout) {
         const self = this;
-        function target() callback.call(self);
+        function target() callback.call(self)
         return window.setTimeout(target, timeout);
     }
 };

diff --git a/common/content/commandline.js b/common/content/commandline.js
@@ -306,7 +306,7 @@ const CommandLine = Module("commandline", {
         liberator.triggerObserver("echoLine", str, highlightGroup, forceSingle);
 
         //if (!this._commandShown())
-            ;//commandline.hide();
+            //commandline.hide();
 
         /*let field = this._messageBox.inputField;
         if (!forceSingle && field.editor.rootElement.scrollWidth > field.scrollWidth)
@@ -1370,7 +1370,7 @@ const CommandLine = Module("commandline", {
                 try {
                     this.waiting = true;
                     for (let context of list) {
-                        function done() !(idx >= n + context.items.length || idx == -2 && !context.items.length);
+                        function done() !(idx >= n + context.items.length || idx == -2 && !context.items.length)
                         while (context.incomplete && !done())
                             liberator.threadYield(false, true);
 
@@ -1799,7 +1799,7 @@ const ItemList = Class("ItemList", {
         let off = 0;
         let end = this._startIndex + options.maxitems;
         function getRows(context) {
-            function fix(n) util.Math.constrain(n, 0, len);
+            function fix(n) util.Math.constrain(n, 0, len)
             let len = context.items.length;
             let start = off;
             end -= !!context.message + context.incomplete;

diff --git a/common/content/commands.js b/common/content/commands.js
@@ -381,7 +381,7 @@ const Commands = Module("commands", {
      */
     commandToString: function (args) {
         let res = [args.command + (args.bang ? "!" : "")];
-        function quote(str) Commands.quoteArg[/[\s"'\\]|^$/.test(str) ? '"' : ""](str);
+        function quote(str) Commands.quoteArg[/[\s"'\\]|^$/.test(str) ? '"' : ""](str)
 
         for (let [opt, val] in Iterator(args.options || {})) {
             let chr = /^-.$/.test(opt) ? " " : "=";
@@ -1214,9 +1214,9 @@ const Commands = Module("commands", {
     function quote(q, list) {
         let re = RegExp("[" + list + "]", "g");
         return function (str) q + String.replace(str, re, function ($0) $0 in Commands.quoteMap ? Commands.quoteMap[$0] : ("\\" + $0)) + q;
-    };
+    }
     function vimSingleQuote(s)
-        s.replace(/'/g, "''");
+        s.replace(/'/g, "''")
     Commands.complQuote = { // FIXME
         '"': ['"', quote("", '\n\t"\\\\'), '"'],
         "'": ["'", vimSingleQuote, "'"],

diff --git a/common/content/hints.js b/common/content/hints.js
@@ -33,9 +33,9 @@ const Hints = Module("hints", {
 
         const Mode = Hints.Mode;
         Mode.defaultValue("tags", function () function () options.hinttags);
-        function extended() options.extendedhinttags;
-        function images() "//*[@src]";
-        function anchors() "//*[@id or @name]";
+        function extended() options.extendedhinttags
+        function images() "//*[@src]"
+        function anchors() "//*[@id or @name]"
 
         this._hintModes = {
             ";": Mode("Focus hint",                         function (elem) buffer.focusElement(elem),                             extended),
@@ -698,7 +698,7 @@ const Hints = Module("hints", {
          * @param {string} str The string to split.
          * @returns {Array(string)} The lowercased splits of the splitting.
          */
-        function tokenize(pat, str) str.split(pat).map(String.toLowerCase);
+        function tokenize(pat, str) str.split(pat).map(String.toLowerCase)
 
         /**
          * Get a hint matcher for hintmatching=contains

diff --git a/common/content/io.js b/common/content/io.js
@@ -254,7 +254,7 @@ const File = Class("File", {
             !liberator.has("Windows") ? /\$(\w+)\b|\${(\w+)}/g
                                  : /\$(\w+)\b|\${(\w+)}|%(\w+)%/g,
             function (m, n1, n2, n3) services.get("environment").get(n1 || n2 || n3) || m
-        );
+        )
         path = expand(path);
 
         // expand ~
@@ -770,7 +770,7 @@ lookup:
     system: function (command, input) {
         liberator.echomsg("Executing: " + command);
 
-        function escape(str) '"' + str.replace(/[\\"$]/g, "\\$&") + '"';
+        function escape(str) '"' + str.replace(/[\\"$]/g, "\\$&") + '"'
 
         return this.withTempFiles(function (stdin, stdout, cmd) {
             if (input)

diff --git a/common/content/javascript.js b/common/content/javascript.js
@@ -383,7 +383,7 @@ const JavaScript = Module("javascript", {
                 //  Constants are unsorted, and appear before other non-null strings.
                 //  Other strings are sorted in the default manner.
                 let compare = context.compare;
-                function isnan(item) item != '' && isNaN(item);
+                function isnan(item) item != '' && isNaN(item)
                 context.compare = function (a, b) {
                     if (!isnan(a.item.key) && !isnan(b.item.key))
                         return a.item.key - b.item.key;

diff --git a/common/content/liberator.js b/common/content/liberator.js
@@ -512,7 +512,7 @@ const Liberator = Module("liberator", {
         let items = completion._runCompleter("help", topic, null, unchunked).items;
         let partialMatch = null;
 
-        function format(item) item.description + "#" + encodeURIComponent(item.text);
+        function format(item) item.description + "#" + encodeURIComponent(item.text)
 
         for (let item of items) {
             if (item.text == topic)
@@ -1470,7 +1470,7 @@ const Liberator = Module("liberator", {
                             e.original.uninstall();
                         else
                             e.original.userDisabled = command.action == "disableItem";
-                    };
+                    }
 
                     if (args.bang)
                         liberator.extensions.forEach(function (e) { action(e); });

diff --git a/common/content/mappings.js b/common/content/mappings.js
@@ -116,7 +116,7 @@ const Map = Class("Map", {
             args.push(argument);
 
         let self = this;
-        function repeat() self.action.apply(self, args);
+        function repeat() self.action.apply(self, args)
         if (this.names[0] != ".") // FIXME: Kludge.
             mappings.repeat = repeat;
 

diff --git a/common/content/modules.js b/common/content/modules.js
@@ -69,7 +69,7 @@ Module.constructors = {};
 window.addEventListener("load", function onload() {
     window.removeEventListener("load", onload, false);
 
-    function dump(str) window.dump(String.replace(str, /\n?$/, "\n").replace(/^/m, Config.prototype.name.toLowerCase() + ": "));
+    function dump(str) window.dump(String.replace(str, /\n?$/, "\n").replace(/^/m, Config.prototype.name.toLowerCase() + ": "))
     const start = Date.now();
     const deferredInit = { load: [] };
     const seen = new Set();
@@ -91,7 +91,7 @@ window.addEventListener("load", function onload() {
             loaded.push(module.name);
 
             function init(mod, module)
-                function () module.INIT[mod].call(modules[module.name], modules[mod]);
+                function () module.INIT[mod].call(modules[module.name], modules[mod])
             for (let mod of loaded) {
                 try {
                     if (mod in module.INIT)

diff --git a/common/content/options.js b/common/content/options.js
@@ -618,7 +618,7 @@ const Options = Module("options", {
                     option.value = xml`=${template.highlight(opt.value)}`;
                 yield option;
             }
-        };
+        }
 
         let list = template.options("Options", opts());
         commandline.echo(list, commandline.HL_NORMAL, commandline.FORCE_MULTILINE);
@@ -658,7 +658,7 @@ const Options = Module("options", {
 
                 yield option;
             }
-        };
+        }
 
         let list = template.options(config.hostApplication + " Options", prefs());
         commandline.echo(list, commandline.HL_NORMAL, commandline.FORCE_MULTILINE);

diff --git a/muttator/components/about-handler.js b/muttator/components/about-handler.js
@@ -39,6 +39,6 @@ AboutHandler.prototype = {
 if(XPCOMUtils.generateNSGetFactory)
     var NSGetFactory = XPCOMUtils.generateNSGetFactory([AboutHandler]);
 else
-    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([AboutHandler]);
+    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([AboutHandler])
 
 // vim: set fdm=marker sw=4 ts=4 et:
diff --git a/muttator/components/commandline-handler.js b/muttator/components/commandline-handler.js
@@ -46,6 +46,6 @@ CommandLineHandler.prototype = {
 if(XPCOMUtils.generateNSGetFactory)
     var NSGetFactory = XPCOMUtils.generateNSGetFactory([CommandLineHandler]);
 else
-    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([CommandLineHandler]);
+    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([CommandLineHandler])
 
 // vim: set ft=javascript fdm=marker sw=4 ts=4 et:
diff --git a/vimperator/components/about-handler.js b/vimperator/components/about-handler.js
@@ -39,6 +39,6 @@ AboutHandler.prototype = {
 if(XPCOMUtils.generateNSGetFactory)
     var NSGetFactory = XPCOMUtils.generateNSGetFactory([AboutHandler]);
 else
-    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([AboutHandler]);
+    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([AboutHandler])
 
 // vim: set fdm=marker sw=4 ts=4 et:
diff --git a/vimperator/components/commandline-handler.js b/vimperator/components/commandline-handler.js
@@ -46,6 +46,6 @@ CommandLineHandler.prototype = {
 if(XPCOMUtils.generateNSGetFactory)
     var NSGetFactory = XPCOMUtils.generateNSGetFactory([CommandLineHandler]);
 else
-    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([CommandLineHandler]);
+    function NSGetModule(compMgr, fileSpec) XPCOMUtils.generateModule([CommandLineHandler])
 
 // vim: set ft=javascript fdm=marker sw=4 ts=4 et:
diff --git a/vimperator/regressions.js b/vimperator/regressions.js
@@ -83,20 +83,20 @@ function resetEnvironment() {
     modes.reset();
 }
 
-function getOutput()            multilineOutput.contentDocument.body.textContent || singlelineOutput.value;
-function getMultilineOutput()   multilineOutput.contentDocument.body.textContent;
-function getSinglelineOutput()  singlelineOutput.value;
+function getOutput()            multilineOutput.contentDocument.body.textContent || singlelineOutput.value
+function getMultilineOutput()   multilineOutput.contentDocument.body.textContent
+function getSinglelineOutput()  singlelineOutput.value
 
-function getTabIndex() getBrowser().mTabContainer.selectedIndex;
-function getTabCount() getBrowser().mTabs.length;
+function getTabIndex() getBrowser().mTabContainer.selectedIndex
+function getTabCount() getBrowser().mTabs.length
 
 function getBufferPosition() {
     let win = window.content;
     return { x: win.scrollMaxX ? win.pageXOffset / win.scrollMaxX : 0,
              y: win.scrollMaxY ? win.pageYOffset / win.scrollMaxY : 0 }
-};
+}
 
-function getLocation() window.content.document.location.href;
+function getLocation() window.content.document.location.href
 
 function echoLine(str, group) {
     if (!doc)