Replace uses of map with list comprehension or generator expressions

Author: jdufresne

debug_toolbar/panels/sql/tracking.py

@@ -93,7 +93,7 @@ def _quote_params(self, params):
         if isinstance(params, dict):
             return dict((key, self._quote_expr(value))
                         for key, value in params.items())
-        return list(map(self._quote_expr, params))
+        return [self._quote_expr(p) for p in params]
 
     def _decode(self, param):
         try:
@@ -114,7 +114,7 @@ def _record(self, method, sql, params):
                 stacktrace = []
             _params = ''
             try:
-                _params = json.dumps(list(map(self._decode, params)))
+                _params = json.dumps([self._decode(p) for p in params])
             except Exception:
                 pass  # object not JSON serializable
 

debug_toolbar/utils.py

@@ -5,6 +5,7 @@
 import re
 import sys
 from importlib import import_module
+from itertools import chain
 
 import django
 from django.core.exceptions import ImproperlyConfigured
@@ -71,7 +72,7 @@ def tidy_stacktrace(stack):
 def render_stacktrace(trace):
     stacktrace = []
     for frame in trace:
-        params = map(escape, frame[0].rsplit(os.path.sep, 1) + list(frame[1:]))
+        params = (escape(v) for v in chain(frame[0].rsplit(os.path.sep, 1), frame[1:]))
         params_dict = dict((six.text_type(idx), v) for idx, v in enumerate(params))
         try:
             stacktrace.append('<span class="djdt-path">%(0)s/</span>'