Merge pull request #556 from a-detiste/master

trim old Python2 compatibility layer

Author: greyli

docs/conf.py

@@ -12,8 +12,6 @@
 # serve to show the default.
 
 import sys, os
-from webassets.six.moves import map
-from webassets.six.moves import zip
 
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the

examples/appengine/helloworld.py

@@ -1,8 +1,3 @@
-from __future__ import print_function
-from __future__ import print_function
-from __future__ import print_function
-from __future__ import print_function
-from __future__ import print_function
 # Import assets configuration
 from assets import bundle
 

src/webassets/bundle.py

@@ -2,8 +2,6 @@
 import os
 from os import path
 from webassets import six
-from webassets.six.moves import map
-from webassets.six.moves import zip
 
 from .filter import get_filter
 from .merge import (FileHunk, UrlHunk, FilterTool, merge, merge_filters,
@@ -161,12 +159,9 @@ def _set_filters(self, value):
             self._filters = ()
             return
 
-        if isinstance(value, six.string_types):
+        if isinstance(value, str):
             # 333: Simplify w/o condition?
-            if six.PY3:
-                filters = map(str.strip, value.split(','))
-            else:
-                filters = map(unicode.strip, unicode(value).split(','))
+            filters = map(str.strip, value.split(','))
         elif isinstance(value, (list, tuple)):
             filters = value
         else:
@@ -276,7 +271,7 @@ def _filter_duplicates(resolved):
     def _get_depends(self):
         return self._depends
     def _set_depends(self, value):
-        self._depends = [value] if isinstance(value, six.string_types) else value
+        self._depends = [value] if isinstance(value, str) else value
         self._resolved_depends = None
     depends = property(_get_depends, _set_depends, doc=
     """Allows you to define an additional set of files (glob syntax

src/webassets/env.py

@@ -1,8 +1,6 @@
 import os
 from os import path
 from itertools import chain
-from webassets import six
-from webassets.six.moves import map
 from webassets.utils import is_url
 
 try:
@@ -242,7 +240,7 @@ def resolve_source(self, ctx, item):
         """
 
         # Pass through some things unscathed
-        if not isinstance(item, six.string_types):
+        if not isinstance(item, str):
             # Don't stand in the way of custom values.
             return item
         if is_url(item) or path.isabs(item):
@@ -308,7 +306,7 @@ def __init__(self):
         self._anon_bundles = []
 
     def __iter__(self):
-        return chain(six.itervalues(self._named_bundles), self._anon_bundles)
+        return chain(self._named_bundles.values(), self._anon_bundles)
 
     def __getitem__(self, name):
         return self._named_bundles[name]

src/webassets/ext/jinja2.py

@@ -1,5 +1,3 @@
-from __future__ import absolute_import
-
 import warnings
 import jinja2
 from jinja2.ext import Extension

src/webassets/filter/__init__.py

@@ -2,17 +2,13 @@
 contents (think minification, compression).
 """
 
-from __future__ import with_statement
-
 import os
 import subprocess
 import inspect
 import shlex
 import tempfile
 import pkgutil
 from webassets import six
-from webassets.six.moves import map
-from webassets.six.moves import zip
 try:
     frozenset
 except NameError:
@@ -34,7 +30,7 @@ def freezedicts(obj):
     if isinstance(obj, (list, tuple)):
         return type(obj)([freezedicts(sub) for sub in obj])
     if isinstance(obj, dict):
-        return frozenset(six.iteritems(obj))
+        return frozenset(obj.items())
     return obj
 
 
@@ -50,19 +46,11 @@ def smartsplit(string, sep):
     be done.
     """
     assert string is not None   # or shlex will read from stdin
-    if not six.PY3:
-        # On 2.6, shlex fails miserably with unicode input
-        is_unicode = isinstance(string, unicode)
-        if is_unicode:
-            string = string.encode('utf8')
     l = shlex.shlex(string, posix=True)
     l.whitespace += ','
     l.whitespace_split = True
     l.quotes = ''
-    if not six.PY3 and is_unicode:
-        return map(lambda s: s.decode('utf8'), list(l))
-    else:
-        return list(l)
+    return list(l)
 
 
 class option(tuple):
@@ -204,9 +192,6 @@ def get_config(self, setting=False, env=None, require=True,
         if value is None and not env is False:
             value = os.environ.get(env)
             if value is not None:
-                if not six.PY3:
-                    # TODO: What charset should we use? What does Python 3 use?
-                    value = value.decode('utf8')
                 if type == list:
                     value = smartsplit(value, ',')
 
@@ -623,7 +608,7 @@ def get_filter(f, *args, **kwargs):
         # Don't need to do anything.
         assert not args and not kwargs
         return f
-    elif isinstance(f, six.string_types):
+    elif isinstance(f, str):
         if f in _FILTERS:
             klass = _FILTERS[f]
         else:

src/webassets/filter/autoprefixer.py

@@ -1,5 +1,3 @@
-from __future__ import with_statement
-
 from webassets.filter import ExternalTool
 from webassets.utils import working_directory
 

src/webassets/filter/clevercss.py

@@ -1,4 +1,3 @@
-from __future__ import absolute_import
 from webassets.filter import Filter
 
 

src/webassets/filter/closure.py

@@ -30,7 +30,6 @@
         ['--output_wrapper', 'foo: %output%']
 """
 
-from __future__ import absolute_import
 from webassets.filter import JavaTool
 
 

src/webassets/filter/coffeescript.py

@@ -1,4 +1,3 @@
-from __future__ import print_function
 import os, subprocess
 
 from webassets.filter import Filter