Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(393)

Unified Diff: third_party/google-endpoints/libfuturize/fixes/fix_remove_old__future__imports.py

Issue 2666783008: Add google-endpoints to third_party/. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/google-endpoints/libfuturize/fixes/fix_remove_old__future__imports.py
diff --git a/third_party/google-endpoints/libfuturize/fixes/fix_remove_old__future__imports.py b/third_party/google-endpoints/libfuturize/fixes/fix_remove_old__future__imports.py
new file mode 100644
index 0000000000000000000000000000000000000000..060eb004176d4a0d856f76a7cb4bc78b3c8164a9
--- /dev/null
+++ b/third_party/google-endpoints/libfuturize/fixes/fix_remove_old__future__imports.py
@@ -0,0 +1,27 @@
+"""
+Fixer for removing any of these lines:
+
+ from __future__ import with_statement
+ from __future__ import nested_scopes
+ from __future__ import generators
+
+The reason is that __future__ imports like these are required to be the first
+line of code (after docstrings) on Python 2.6+, which can get in the way.
+
+These imports are always enabled in Python 2.6+, which is the minimum sane
+version to target for Py2/3 compatibility.
+"""
+
+from lib2to3 import fixer_base
+from libfuturize.fixer_util import remove_future_import
+
+class FixRemoveOldFutureImports(fixer_base.BaseFix):
+ BM_compatible = True
+ PATTERN = "file_input"
+ run_order = 1
+
+ def transform(self, node, results):
+ remove_future_import(u"with_statement", node)
+ remove_future_import(u"nested_scopes", node)
+ remove_future_import(u"generators", node)
+

Powered by Google App Engine
This is Rietveld 408576698