From 3ae51ed14c1139ee026d411577d5b349a215a075 Mon Sep 17 00:00:00 2001
From: Jon Dufresne <jon.dufresne@gmail.com>
Date: Fri, 5 Jan 2018 08:39:20 -0800
Subject: [PATCH] Update uses of map() to use list/set comprehensions instead

Updates style to modern Python idioms.
---
 Lib/ldap/controls/deref.py |  2 +-
 Lib/ldap/filter.py         |  2 +-
 Lib/ldap/functions.py      |  3 +--
 Lib/ldap/modlist.py        | 10 +++++-----
 Lib/ldapurl.py             |  2 +-
 5 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/Lib/ldap/controls/deref.py b/Lib/ldap/controls/deref.py
index b927f1a..0d43ab1 100644
--- a/Lib/ldap/controls/deref.py
+++ b/Lib/ldap/controls/deref.py
@@ -108,7 +108,7 @@ def decodeControlValue(self,encodedControlValue):
     for deref_res in decodedValue:
       deref_attr,deref_val,deref_vals = deref_res[0],deref_res[1],deref_res[2]
       partial_attrs_dict = {
-        str(tv[0]): map(str,tv[1])
+        str(tv[0]): [str(v) for v in tv[1]]
         for tv in deref_vals or []
       }
       try:
diff --git a/Lib/ldap/filter.py b/Lib/ldap/filter.py
index 56665dc..3dba7f7 100644
--- a/Lib/ldap/filter.py
+++ b/Lib/ldap/filter.py
@@ -54,7 +54,7 @@ def filter_format(filter_template,assertion_values):
         List or tuple of assertion values. Length must match
         count of %s in filter_template.
   """
-  return filter_template % (tuple(map(escape_filter_chars,assertion_values)))
+  return filter_template % tuple(escape_filter_chars(v) for v in assertion_values)
 
 
 def time_span_filter(
diff --git a/Lib/ldap/functions.py b/Lib/ldap/functions.py
index 8c7580e..6c351ef 100644
--- a/Lib/ldap/functions.py
+++ b/Lib/ldap/functions.py
@@ -138,8 +138,7 @@ def escape_str(escape_func,s,*args):
   Applies escape_func() to all items of `args' and returns a string based
   on format string `s'.
   """
-  escape_args = map(escape_func,args)
-  return s % tuple(escape_args)
+  return s % tuple(escape_func(v) for v in args)
 
 
 def strf_secs(secs):
diff --git a/Lib/ldap/modlist.py b/Lib/ldap/modlist.py
index ecd7a42..6f51312 100644
--- a/Lib/ldap/modlist.py
+++ b/Lib/ldap/modlist.py
@@ -11,7 +11,7 @@
 
 def addModlist(entry,ignore_attr_types=None):
   """Build modify list for call of method LDAPObject.add()"""
-  ignore_attr_types = set(map(str.lower,ignore_attr_types or []))
+  ignore_attr_types = {v.lower() for v in ignore_attr_types or []}
   modlist = []
   for attrtype in entry.keys():
     if attrtype.lower() in ignore_attr_types:
@@ -46,8 +46,8 @@ def modifyModlist(
       List of attribute type names for which comparison will be made
       case-insensitive
   """
-  ignore_attr_types = set(map(str.lower,ignore_attr_types or []))
-  case_ignore_attr_types = set(map(str.lower,case_ignore_attr_types or []))
+  ignore_attr_types = {v.lower() for v in ignore_attr_types or []}
+  case_ignore_attr_types = {v.lower() for v in case_ignore_attr_types or []}
   modlist = []
   attrtype_lower_map = {}
   for a in old_entry.keys():
@@ -73,8 +73,8 @@ def modifyModlist(
       replace_attr_value = len(old_value)!=len(new_value)
       if not replace_attr_value:
         if attrtype_lower in case_ignore_attr_types:
-          old_value_set = set(map(str.lower,old_value))
-          new_value_set = set(map(str.lower,new_value))
+          old_value_set = {v.lower() for v in old_value}
+          new_value_set = {v.lower() for v in new_value}
         else:
           old_value_set = set(old_value)
           new_value_set = set(new_value)
diff --git a/Lib/ldapurl.py b/Lib/ldapurl.py
index 374336d..698889a 100644
--- a/Lib/ldapurl.py
+++ b/Lib/ldapurl.py
@@ -158,7 +158,7 @@ def values(self):
     ]
 
   def __str__(self):
-    return ','.join(map(str,self.values()))
+    return ','.join(str(v) for v in self.values())
 
   def __repr__(self):
     return '<%s.%s instance at %s: %s>' % (