From 1d84edc0b2fce08034316c1665b6dc75d32d9ee6 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Sun, 4 Feb 2018 08:57:32 +0000
Subject: Remove \I (ignore case) flag for find mode.

It appears that nobody is using this (and it is broken).  So this PR
removes this feature.

Fixes #2937 (kind of).
---
 content_scripts/mode_find.coffee | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/content_scripts/mode_find.coffee b/content_scripts/mode_find.coffee
index f19b5db4..5090e4ee 100644
--- a/content_scripts/mode_find.coffee
+++ b/content_scripts/mode_find.coffee
@@ -104,7 +104,6 @@ class FindMode extends Mode
     # escape sequences. '\' is the escape character and needs to be escaped itself to be used as a normal
     # character. here we grep for the relevant escape sequences.
     @query.isRegex = Settings.get 'regexFindMode'
-    hasNoIgnoreCaseFlag = false
     @query.parsedQuery = @query.rawQuery.replace /(\\{1,2})([rRI]?)/g, (match, slashes, flag) =>
       return match if flag == "" or slashes.length != 1
       switch (flag)
@@ -112,12 +111,10 @@ class FindMode extends Mode
           @query.isRegex = true
         when "R"
           @query.isRegex = false
-        when "I"
-          hasNoIgnoreCaseFlag = true
       ""
 
-    # default to 'smartcase' mode, unless noIgnoreCase is explicitly specified
-    @query.ignoreCase = !hasNoIgnoreCaseFlag && !Utils.hasUpperCase(@query.parsedQuery)
+    # Implement smartcase.
+    @query.ignoreCase = not Utils.hasUpperCase(@query.parsedQuery)
 
     regexPattern = if @query.isRegex
       @query.parsedQuery
-- 
cgit v1.2.3