refactor: simplify code and avoid unnecessary array allocations

Author: 9romise

lib/rules/no-multiple-template-root.js

@@ -6,18 +6,6 @@
 
 const utils = require('../utils')
 
-/**
- * Get all comments that need to be reported
- * @param {(HTMLComment | HTMLBogusComment | Comment)[]} comments
- * @param {Range[]} elementRanges
- * @returns {(HTMLComment | HTMLBogusComment | Comment)[]}
- */
-function getReportComments(comments, elementRanges) {
-  return comments.filter(
-    (comment) => !elementRanges.some((range) => utils.inRange(range, comment))
-  )
-}
-
 module.exports = {
   meta: {
     type: 'problem',
@@ -65,9 +53,12 @@ module.exports = {
         const comments = element.comments.filter((comment) =>
           utils.inRange(element, comment)
         )
-        const elementRanges = element.children.map((child) => child.range)
-        if (disallowComments && comments.length > 0) {
-          for (const comment of getReportComments(comments, elementRanges)) {
+        const reportComments = comments.filter(
+          (comment) =>
+            !element.children.some((child) => utils.inRange(child, comment))
+        )
+        if (disallowComments) {
+          for (const comment of reportComments) {
             context.report({
               node: comment,
               loc: comment.loc,