summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorClaudiu Popa <cpopa@cloudbasesolutions.com>2014-11-17 21:28:55 +0200
committerClaudiu Popa <cpopa@cloudbasesolutions.com>2014-11-17 21:28:55 +0200
commitfd9acc81a549e352243fbdf55eee270f98b07dfa (patch)
treeb1e44240515b225b611b61b0c632aebc98dc2dab
parente6ca1997bca58f3962a1991870e58f61d44cbeff (diff)
downloadpylint-git-fd9acc81a549e352243fbdf55eee270f98b07dfa.tar.gz
Fix the flawed check of old octal syntax.
-rw-r--r--checkers/python3.py2
-rw-r--r--test/unittest_checker_python3.py5
2 files changed, 6 insertions, 1 deletions
diff --git a/checkers/python3.py b/checkers/python3.py
index 1da293316..da8f6c470 100644
--- a/checkers/python3.py
+++ b/checkers/python3.py
@@ -23,7 +23,7 @@ from pylint.utils import WarningScope
from pylint.checkers import utils
-_OLD_OCTAL = re.compile("\d{2}")
+_OLD_OCTAL = re.compile("0\d{1}")
def _is_old_octal(literal):
return _OLD_OCTAL.match(literal)
diff --git a/test/unittest_checker_python3.py b/test/unittest_checker_python3.py
index f7eead957..4f9e2ab82 100644
--- a/test/unittest_checker_python3.py
+++ b/test/unittest_checker_python3.py
@@ -285,6 +285,11 @@ class Python3TokenCheckerTest(testutils.CheckerTestCase):
def test_old_octal_literal(self):
self._test_token_message("045", "old-octal-literal")
+ # Make sure we are catching only octals.
+ tokens = testutils.tokenize_str("45")
+ with self.assertNoMessages():
+ self.checker.process_tokens(tokens)
+
if __name__ == '__main__':
unittest.main()