diff options
author | Florent Xicluna <florent.xicluna@gmail.com> | 2014-04-26 22:37:04 +0200 |
---|---|---|
committer | Florent Xicluna <florent.xicluna@gmail.com> | 2014-04-26 22:37:04 +0200 |
commit | 7b9449d525bcc2c5df4699616d8b75de76a586d4 (patch) | |
tree | 2c1ba89119efd3a98a5a9e829fccdc0f8196fe07 | |
parent | a176b77c22ef0c0263ca4f956b9b7c91ea659885 (diff) | |
download | pep8-7b9449d525bcc2c5df4699616d8b75de76a586d4.tar.gz |
Replace codes E111/2/3 with E114/5/6 for indentation of comments; issue #274
-rw-r--r-- | CHANGES.txt | 3 | ||||
-rw-r--r-- | docs/intro.rst | 6 | ||||
-rwxr-xr-x | pep8.py | 15 | ||||
-rw-r--r-- | testsuite/E11.py | 22 |
4 files changed, 40 insertions, 6 deletions
diff --git a/CHANGES.txt b/CHANGES.txt index e12f89d..4c32dfa 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -9,6 +9,9 @@ Bug fixes: * Skip the traceback on "Broken pipe" signal. (Issue #275) +* Replace codes E111, E112 and E113 with codes E114, E115 and E116 + for wrong indentation of comments. (Issue #274) + 1.5.6 (2014-04-14) ------------------ diff --git a/docs/intro.rst b/docs/intro.rst index 018521d..06b44f0 100644 --- a/docs/intro.rst +++ b/docs/intro.rst @@ -193,6 +193,12 @@ This is the current list of error and warning codes: +----------+----------------------------------------------------------------------+ | E113 | unexpected indentation | +----------+----------------------------------------------------------------------+ +| E114 | indentation is not a multiple of four (comment) | ++----------+----------------------------------------------------------------------+ +| E115 | expected an indented block (comment) | ++----------+----------------------------------------------------------------------+ +| E116 | unexpected indentation (comment) | ++----------+----------------------------------------------------------------------+ +----------+----------------------------------------------------------------------+ | E121 (^) | continuation line under-indented for hanging indent | +----------+----------------------------------------------------------------------+ @@ -353,20 +353,25 @@ def indentation(logical_line, previous_logical, indent_char, Okay: a = 1 Okay: if a == 0:\n a = 1 E111: a = 1 + E114: # a = 1 Okay: for item in items:\n pass E112: for item in items:\npass + E115: for item in items:\n# Hi\n pass Okay: a = 1\nb = 2 E113: a = 1\n b = 2 + E116: a = 1\n # b = 2 """ - if indent_char == ' ' and indent_level % 4: - yield 0, "E111 indentation is not a multiple of four" + c = 0 if logical_line else 3 + tmpl = "E11%d %s" if logical_line else "E11%d %s (comment)" + if indent_level % 4: + yield 0, tmpl % (1 + c, "indentation is not a multiple of four") indent_expect = previous_logical.endswith(':') if indent_expect and indent_level <= previous_indent_level: - yield 0, "E112 expected an indented block" - if indent_level > previous_indent_level and not indent_expect: - yield 0, "E113 unexpected indentation" + yield 0, tmpl % (2 + c, "expected an indented block") + elif not indent_expect and indent_level > previous_indent_level: + yield 0, tmpl % (3 + c, "unexpected indentation") def continued_indentation(logical_line, tokens, indent_level, hang_closing, diff --git a/testsuite/E11.py b/testsuite/E11.py index 98b9431..4ed1096 100644 --- a/testsuite/E11.py +++ b/testsuite/E11.py @@ -10,7 +10,27 @@ print #: E113 print print -#: E111 E113 +#: E114 E116 mimetype = 'application/x-directory' # 'httpd/unix-directory' create_date = False +#: E116 E116 E116 +def start(self): + if True: + self.master.start() + # try: + # self.master.start() + # except MasterExit: + # self.shutdown() + # finally: + # sys.exit() +#: E115 E115 E115 E115 E115 E115 +def start(self): + if True: +# try: +# self.master.start() +# except MasterExit: +# self.shutdown() +# finally: +# sys.exit() + self.master.start() |