Noissue - Added missing semicolons and other trivial fixes.

lib/subscriptionClasses.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 104 matching lines @@
     if (value != this._disabled)
     {
       let oldValue = this._disabled;
       this._disabled = value;
       FilterNotifier.triggerListeners("subscription.disabled", this, value, oldValue);
     }
     return this._disabled;
   },
 
   /**
-   * Serializes the filter to an array of strings for writing out on the disk.
+   * Serializes the subscription to an array of strings for writing out on the disk.
    * @param {Array of String} buffer  buffer to push the serialization results into
    */
   serialize: function(buffer)
   {
     buffer.push("[Subscription]");
     buffer.push("url=" + this.url);
     buffer.push("title=" + this._title);
     if (this._fixedTitle)
       buffer.push("fixedTitle=true");
     if (this._disabled)
@@ skipping 33 matching lines @@
   try
   {
     // Test URL for validity
     url = Services.io.newURI(url, null, null).spec;
     return new DownloadableSubscription(url, null);
   }
   catch (e)
   {
     return new SpecialSubscription(url);
   }
-}
+};
 
 /**
  * Deserializes a subscription
  *
  * @param {Object}  obj map of serialized properties and their values
  * @return {Subscription} subscription or null if the subscription couldn't be created
  */
 Subscription.fromObject = function(obj)
 {
   let result;
@@ skipping 53 matching lines @@
     result = new SpecialSubscription(obj.url, obj.title);
     if ("defaults" in obj)
       result.defaults = obj.defaults.split(" ");
   }
   if ("fixedTitle" in obj)
     result._fixedTitle = (obj.fixedTitle == "true");
   if ("disabled" in obj)
     result._disabled = (obj.disabled == "true");
 
   return result;
-}
+};
 
 /**
  * Class for special filter subscriptions (user's filters)
  * @param {String} url see Subscription()
  * @param {String} [title]  see Subscription()
  * @constructor
  * @augments Subscription
  */
 function SpecialSubscription(url, title)
 {
@@ skipping 58 matching lines @@
  * @param {String} [title]  title of the new filter group
  * @result {SpecialSubscription}
  */
 SpecialSubscription.create = function(title)
 {
   let url;
   do
   {
     url = "~user~" + Math.round(Math.random()*1000000);
   } while (url in Subscription.knownSubscriptions);
-  return new SpecialSubscription(url, title)
+  return new SpecialSubscription(url, title);
 };
 
 /**
  * Creates a new user-defined filter group and adds the given filter to it.
  * This group will act as the default group for this filter type.
  */
 SpecialSubscription.createForFilter = function(/**Filter*/ filter) /**SpecialSubscription*/
 {
   let subscription = SpecialSubscription.create();
   subscription.filters.push(filter);
@@ skipping 75 matching lines @@
   {
     Subscription.prototype.serialize.call(this, buffer);
     if (this._homepage)
       buffer.push("homepage=" + this._homepage);
     if (this._lastDownload)
       buffer.push("lastDownload=" + this._lastDownload);
   }
 };
 
 /**
- * Class for filter subscriptions updated by externally (by other extension)
+ * Class for filter subscriptions updated externally (by other extension)
  * @param {String} url    see Subscription()
  * @param {String} [title]  see Subscription()
  * @constructor
  * @augments RegularSubscription
  */
 function ExternalSubscription(url, title)
 {
   RegularSubscription.call(this, url, title);
 }
 exports.ExternalSubscription = ExternalSubscription;
 
 ExternalSubscription.prototype =
 {
   __proto__: RegularSubscription.prototype,
 
   /**
    * See Subscription.serialize()
    */
   serialize: function(buffer)
   {
     throw new Error("Unexpected call, external subscriptions should not be serialized");
   }
 };
 
 /**
- * Class for filter subscriptions updated by externally (by other extension)
+ * Class for filter subscriptions updated externally (by other extension)
  * @param {String} url  see Subscription()
  * @param {String} [title]  see Subscription()
  * @constructor
  * @augments RegularSubscription
  */
 function DownloadableSubscription(url, title)
 {
   RegularSubscription.call(this, url, title);
 }
 exports.DownloadableSubscription = DownloadableSubscription;
@@ skipping 90 matching lines @@
    * Minimal Adblock Plus version required for this subscription
    * @type String
    */
   requiredVersion: null,
 
   /**
    * Should be true if requiredVersion is higher than current Adblock Plus version
    * @type Boolean
    */
   upgradeRequired: false,
-  
+
   /**
    * Number indicating how often the object was downloaded.
    * @type Number
    */
   downloadCount: 0,
 
   /**
    * See Subscription.serialize()
    */
   serialize: function(buffer)
@@ skipping 12 matching lines @@
     if (this.errors)
       buffer.push("errors=" + this.errors);
     if (this.version)
       buffer.push("version=" + this.version);
     if (this.requiredVersion)
       buffer.push("requiredVersion=" + this.requiredVersion);
     if (this.downloadCount)
       buffer.push("downloadCount=" + this.downloadCount);
   }
 };