[gedit/gnome-3-8] Remove all uses of map() and filter(), instead use list comprehensions
- From: Garrett Regier <gregier src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gedit/gnome-3-8] Remove all uses of map() and filter(), instead use list comprehensions
- Date: Sat, 15 Jun 2013 10:12:29 +0000 (UTC)
commit 60306ff2a6b48ae6876fb1fab1e4ca76743c264e
Author: Garrett Regier <garrettregier gmail com>
Date: Fri Jun 14 23:50:13 2013 -0700
Remove all uses of map() and filter(), instead use list comprehensions
Also, stop creating a pointless list just for str.join().
plugins/externaltools/tools/library.py | 2 +-
plugins/snippets/snippets/completion.py | 4 ++--
plugins/snippets/snippets/document.py | 2 +-
tools/generate-message.py | 28 +++++++++++++++-------------
4 files changed, 19 insertions(+), 17 deletions(-)
---
diff --git a/plugins/externaltools/tools/library.py b/plugins/externaltools/tools/library.py
index 6efd03d..1e2a1aa 100644
--- a/plugins/externaltools/tools/library.py
+++ b/plugins/externaltools/tools/library.py
@@ -225,7 +225,7 @@ class Tool(object):
if value.strip() == '':
return []
else:
- return map(lambda x: x.strip(), value.split(','))
+ return [x.strip() for x in value.split(',')]
def _from_list(self, value):
return ','.join(value)
diff --git a/plugins/snippets/snippets/completion.py b/plugins/snippets/snippets/completion.py
index 69c7c38..d48a6a8 100644
--- a/plugins/snippets/snippets/completion.py
+++ b/plugins/snippets/snippets/completion.py
@@ -109,9 +109,9 @@ class Provider(GObject.Object, GtkSource.CompletionProvider):
# Filter based on the current word
if word:
- proposals = filter(lambda x: x['tag'].startswith(word), proposals)
+ proposals = (x for x in proposals if x['tag'].startswith(word))
- return list(map(lambda x: Proposal(x), proposals))
+ return [Proposal(x) for x in proposals]
def do_populate(self, context):
proposals = self.get_proposals(self.get_word(context))
diff --git a/plugins/snippets/snippets/document.py b/plugins/snippets/snippets/document.py
index ccaf42d..fb14647 100644
--- a/plugins/snippets/snippets/document.py
+++ b/plugins/snippets/snippets/document.py
@@ -571,7 +571,7 @@ class Document(GObject.Object, Gedit.ViewActivatable, Signals):
self.active_snippets.append(sn)
# Put cursor at first tab placeholder
- keys = list(filter(lambda x: x > 0, sn.placeholders.keys()))
+ keys = [x for x in sn.placeholders.keys() if x > 0]
if len(keys) == 0:
if 0 in sn.placeholders:
diff --git a/tools/generate-message.py b/tools/generate-message.py
index 6eb0b45..7c67cfd 100755
--- a/tools/generate-message.py
+++ b/tools/generate-message.py
@@ -56,18 +56,20 @@ class ParamSpec:
argstr = ''
if self.args:
- ret = (",\n%s" % (indent,)).join(map(lambda x: str(x), self.args))
+ ret = (",\n%s" % (indent,)).join(str(x) for x in self.args)
argstr = "\n%s%s," % (indent, ret)
+ flags = (' |\n%s' % (indent,)).join(x.strip() for x in self.flags.split('|'))
+
return "%s (%s,\n%s%s,\n%s%s,%s\n%s%s)" % (name,
- self.format_str(self.name),
- indent,
- self.format_str(self.nick),
- indent,
- self.format_str(self.desc),
- argstr,
- indent,
- (' |\n%s' % (indent,)).join([x.strip() for x in
self.flags.split('|')]))
+ self.format_str(self.name),
+ indent,
+ self.format_str(self.nick),
+ indent,
+ self.format_str(self.desc),
+ argstr,
+ indent,
+ flags)
def write(self):
delim = "\n" + (" " * 33)
@@ -88,7 +90,7 @@ class ParamSpecTyped(ParamSpec):
parts = kwargs['gtype'].split('_')
del parts[1]
- self._ctype = '%s *' % (''.join([x.title() for x in parts]),)
+ self._ctype = '%s *' % (''.join(x.title() for x in parts),)
self.args.append(kwargs['gtype'])
@@ -406,7 +408,7 @@ class Message:
self.ns_upper = self.namespace.upper()
self.ns_lower = self.namespace.lower()
- self.name_upper = '_'.join([x.upper() for x in parts])
+ self.name_upper = '_'.join(x.upper() for x in parts)
self.name_lower = self.name_upper.lower()
self.cname_upper = '%s_%s' % (self.ns_upper, self.name_upper)
@@ -416,7 +418,7 @@ class Message:
self.cclass = '%s%sClass' % (self.namespace, self.name)
self.cobj = '%s%s' % (self.namespace, self.name)
- self.filename = '%s-%s' % (self.ns_lower, '-'.join([x.lower() for x in parts]))
+ self.filename = '%s-%s' % (self.ns_lower, '-'.join(x.lower() for x in parts))
def normalize_name(self, name):
if not name:
@@ -425,7 +427,7 @@ class Message:
return name.replace('_', '-')
def parse_flags(self, flags):
- return ' | '.join(['G_PARAM_' + x.strip().replace('-', '_').upper() for x in flags.split('|')])
+ return ' | '.join('G_PARAM_' + x.strip().replace('-', '_').upper() for x in flags.split('|'))
def add_prop(self, elem):
name = self.normalize_name(elem.get('name'))
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]