commit 1f8bfb6f1c10b2dc40bd11a073eaa9564516a401
author Paul Wankadia <junyer@google.com> Mon May 11 17:48:40 2015 +1000
committer Paul Wankadia <junyer@google.com> Tue May 12 05:24:47 2015 +0000
tree 3887165f2733cd82a931de74f864422c8d79ea51
parent 8f01f1566a77ba04b15d79475b7a8a797112f4bd

Fix a minor bug in Compiler::PostVisit().

Otherwise, it unnecessarily sets foldcase for [ to ` inclusive.

Change-Id: I0d56347c97f3440d6c469ca84d4296f7d7ade540
Reviewed-on: https://code-review.googlesource.com/2475
Reviewed-by: Paul Wankadia <junyer@google.com>

re2/compile.cc

diff --git a/re2/compile.cc b/re2/compile.cc
index 4d84ef3..6478def 100644
--- a/re2/compile.cc
+++ b/re2/compile.cc
@@ -815,7 +815,8 @@
         // If this range contains all of A-Za-z or none of it,
         // the fold flag is unnecessary; don't bother.
         bool fold = foldascii;
-        if ((i->lo <= 'A' && 'z' <= i->hi) || i->hi < 'A' || 'z' < i->lo)
+        if ((i->lo <= 'A' && 'z' <= i->hi) || i->hi < 'A' || 'z' < i->lo ||
+            ('Z' < i->lo && i->hi < 'a'))
           fold = false;
 
         AddRuneRange(i->lo, i->hi, fold);