Fix false E302 when there's a comment before the first function or class; issue #174

Author: florentx

CHANGES.txt

@@ -17,6 +17,9 @@ Changelog
 
 * Change misleading message for E251. (Issue #171)
 
+* Do not report false E302 when the source file has a coding cookie or a
+  comment on the first line. (Issue #174)
+
 
 1.4.4 (2013-02-24)
 ------------------

pep8.py

@@ -256,7 +256,7 @@ def blank_lines(logical_line, blank_lines, indent_level, line_number,
     E303: def a():\n\n\n\n    pass
     E304: @decorator\n\ndef a():\n    pass
     """
-    if line_number == 1:
+    if line_number < 3 and not previous_logical:
         return  # Don't expect blank lines before the first line
     if previous_logical.startswith('@'):
         if blank_lines:

testsuite/E30.py

@@ -16,6 +16,19 @@ def b():
 #:
 
 
+#: E302
+#!python
+# -*- coding: utf-8 -*-
+def a():
+    pass
+#: E302
+"""Main module."""
+def _main():
+    pass
+#: E302
+import sys
+def get_sys_path():
+    return sys.path
 #: E302
 def a():
     pass

testsuite/E30not.py

@@ -1,3 +1,19 @@
+#: Okay
+class X:
+    pass
+#: Okay
+
+def foo():
+    pass
+#: Okay
+# -*- coding: utf-8 -*-
+class X:
+    pass
+#: Okay
+# -*- coding: utf-8 -*-
+def foo():
+    pass
+#: Okay
 class X:
 
     def a():