Removes unnecessary parentheses and makes precedence of || and && exp…

…licit
yosef-abraham · May 2, 2016 · 7aa53ed · 7aa53ed
1 parent e347a37
commit 7aa53ed
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
@@ -563,8 +563,8 @@ findAndFollowLink = (linkStrings) ->
 
     linkMatches = false
     for linkString in linkStrings
-      if (link.innerText.toLowerCase().indexOf(linkString) != -1 || link.value &&
-          link.value.indexOf(linkString) != -1)
+      if link.innerText.toLowerCase().indexOf(linkString) != -1 ||
+          link.value?.indexOf(linkString) != -1
         linkMatches = true
         break
     continue unless linkMatches
@@ -596,8 +596,8 @@ findAndFollowLink = (linkStrings) ->
       else
         new RegExp linkString, "i"
     for candidateLink in candidateLinks
-      if (exactWordRegex.test(candidateLink.innerText) || 
-          candidateLink.value && exactWordRegex.test(candidateLink.value))
+      if exactWordRegex.test(candidateLink.innerText) ||
+          (candidateLink.value && exactWordRegex.test(candidateLink.value))
         followLink(candidateLink)
         return true
   false