Merge pull request #218 from hit9/fix-215

Add word boundary for true/false literal lexing, close #215
Thriftpy · Jul 21, 2016 · e9c087f · e9c087f
2 parents f13657b + 115b8a8
commit e9c087f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/tests/parser-cases/issue_215.thrift b/tests/parser-cases/issue_215.thrift
@@ -0,0 +1,6 @@
+struct Test {
+    1: required i64 trueNum;
+}
+
+const bool abool = true
+const i32 falseValue = 123
diff --git a/tests/test_parser.py b/tests/test_parser.py
@@ -254,3 +254,9 @@ def test_recursive_union():
         6: (TType.MAP, 'object', (TType.STRING, (TType.STRUCT,
                                                  thrift.Dynamic)), False)
     }
+
+
+def test_issue_215():
+    thrift = load('parser-cases/issue_215.thrift')
+    assert thrift.abool is True
+    assert thrift.falseValue == 123
diff --git a/thriftpy/parser/lexer.py b/thriftpy/parser/lexer.py
@@ -191,7 +191,7 @@ def t_ignore_COMMENT(t):
 
 
 def t_BOOLCONSTANT(t):
-    r'true|false'
+    r'\btrue\b|\bfalse\b'
     t.value = t.value == 'true'
     return t