Switch to arrow functions instead of passing thisArg

Author: danmactough

lib/feedparser.js

@@ -196,21 +196,21 @@ FeedParser.prototype.handleOpenTag = function (node) {
   if (this.in_xhtml && this.xhtml['#name'] != n['#name']) { // We are in an xhtml node
     // This builds the opening tag, e.g., <div id='foo' class='bar'>
     this.xhtml['#'] += '<' + n['#name'];
-    Object.keys(n['@']).forEach(function (name) {
+    Object.keys(n['@']).forEach((name) => {
       this.xhtml['#'] += ' ' + name + '="' + n['@'][name] + '"';
-    }, this);
+    });
     this.xhtml['#'] += '>';
   } else if (this.stack.length === 0 &&
     (n['#name'] === 'rss' ||
       (n['#local'] === 'rdf' && _.nslookup([n['#uri']], 'rdf')) ||
       (n['#local'] === 'feed' && _.nslookup([n['#uri']], 'atom')))) {
-    Object.keys(n['@']).forEach(function (name) {
+    Object.keys(n['@']).forEach((name) => {
       var o = {};
       if (name != 'version') {
         o[name] = n['@'][name];
         this.meta['@'].push(o);
       }
-    }, this);
+    });
     switch (n['#local']) {
     case 'rss':
       this.meta['#type'] = 'rss';
@@ -433,7 +433,7 @@ FeedParser.prototype.handleAttributes = function handleAttributes (attrs, el) {
     basepath = this.xmlbase[0]['#'];
   }
 
-  Object.keys(attrs).forEach(/** @this {FeedParserInstance} */ function (key) {
+  Object.keys(attrs).forEach(/** @type (key: string) => void */ (key) => {
     var attr = attrs[key]
       , ns = {}
       , prefix = ''
@@ -469,7 +469,7 @@ FeedParser.prototype.handleAttributes = function handleAttributes (attrs, el) {
       this.xhtml = { '#name': el, '#': '' };
     }
     simplifiedAttributes[prefix + attr.local] = attr.value ? attr.value.trim() : '';
-  }, this);
+  });
   return simplifiedAttributes;
 };
 
@@ -496,7 +496,7 @@ FeedParser.prototype.handleMeta = function handleMeta (node, type, options) {
     meta.categories = [];
   }
 
-  Object.keys(node).forEach(function (name) {
+  Object.keys(node).forEach((name) => {
     var el = node[name];
 
     if (normalize) {
@@ -525,7 +525,7 @@ FeedParser.prototype.handleMeta = function handleMeta (node, type, options) {
       case ('atom:link'):
       case ('atom10:link'):
         if (Array.isArray(el)) {
-          el.forEach(function (link) {
+          el.forEach((link) => {
             if (link['@']['href']) { // Atom
               if (_.get(link['@'], 'rel')) {
                 if (link['@']['rel'] == 'alternate') {
@@ -558,7 +558,7 @@ FeedParser.prototype.handleMeta = function handleMeta (node, type, options) {
             else if (this.xmlbase && this.xmlbase.length > 0) {
               meta.link = _.resolve(_.get(this.xmlbase[0], '#'), meta.link);
             }
-          }, this);
+          });
         } else {
           if (el['@']['href']) { // Atom
             if (_.get(el['@'], 'rel')) {
@@ -762,7 +762,7 @@ FeedParser.prototype.handleMeta = function handleMeta (node, type, options) {
       if (~name.indexOf(':')) meta[name] = el;
       else meta[type + ':' + name] = el;
     }
-  }, this); // forEach end
+  }); // forEach end
 
   if (normalize) {
     if (!meta.description) {
@@ -851,7 +851,7 @@ FeedParser.prototype.handleItem = function handleItem (node, type, options) {
     item.enclosures = [];
   }
 
-  Object.keys(node).forEach(function (name) {
+  Object.keys(node).forEach((name) => {
     var el = node[name]
       , attrs = _.get(el, '@')
       , enclosure;
@@ -1135,7 +1135,7 @@ FeedParser.prototype.handleItem = function handleItem (node, type, options) {
       if (~name.indexOf(':')) item[name] = el;
       else item[type + ':' + name] = el;
     }
-  }, this); // forEach end
+  }); // forEach end
 
   if (normalize) {
     if (!item.description) {