From: "André Erdmann" <dywi@mailerd.de>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] proj/R_overlay:master commit in: roverlay/
Date: Wed, 6 Jun 2012 19:52:35 +0000 (UTC) [thread overview]
Message-ID: <1339012078.1f51ac4acac24633cba9d7a531c7165af7acd09d.dywi@gentoo> (raw)
commit: 1f51ac4acac24633cba9d7a531c7165af7acd09d
Author: André Erdmann <dywi <AT> mailerd <DOT> de>
AuthorDate: Wed Jun 6 19:47:58 2012 +0000
Commit: André Erdmann <dywi <AT> mailerd <DOT> de>
CommitDate: Wed Jun 6 19:47:58 2012 +0000
URL: http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=1f51ac4a
descriptionfields.py: reduce function calls
modified: roverlay/descriptionfields.py
---
roverlay/descriptionfields.py | 60 +++++++++++++++++------------------------
1 files changed, 25 insertions(+), 35 deletions(-)
diff --git a/roverlay/descriptionfields.py b/roverlay/descriptionfields.py
index 72175cb..710f0c0 100644
--- a/roverlay/descriptionfields.py
+++ b/roverlay/descriptionfields.py
@@ -19,6 +19,11 @@ class DescriptionField:
self.name = name
+ self.default_value = None
+ self.flags = list ()
+ self.allowed_values = list ()
+ self.aliases = dict ()
+
# --- end of __init__ (...) ---
@@ -36,10 +41,7 @@ class DescriptionField:
arguments:
* flag -- name of the flag
"""
- if not hasattr ( self, 'flags' ):
- self.flags = set ()
-
- self.flags.add ( flag.lower() )
+ self.flags.append ( flag.lower() )
return None
@@ -55,10 +57,7 @@ class DescriptionField:
* value -- allowed value
"""
- if not hasattr ( self, 'allowed_values' ):
- self.allowed_values = set ()
-
- self.allowed_values.add ( value )
+ self.allowed_values.append ( value )
return None
@@ -69,8 +68,7 @@ class DescriptionField:
"""Removes a flag from this DescriptionField. Does nothing if the flag
does not exist.
"""
- if hasattr ( self, 'flags' ):
- self.flags.discard ( flag.lower() )
+ self.flags.discard ( flag.lower() )
return None
# --- end of del_flag (...) ---
@@ -90,8 +88,6 @@ class DescriptionField:
raises: KeyError if alias_type unknown.
"""
- if not hasattr ( self, 'aliases' ):
- self.aliases = dict ()
to_add = dict (
withcase = alias,
@@ -100,9 +96,9 @@ class DescriptionField:
if not alias_type in self.aliases:
- self.aliases [alias_type] = set ()
+ self.aliases [alias_type] = list ()
- self.aliases [alias_type] . add ( to_add )
+ self.aliases [alias_type] . append ( to_add )
return None
@@ -128,7 +124,7 @@ class DescriptionField:
"""Returns the default value for this DescriptionField if it exists,
else None.
"""
- return self.default_value if hasattr ( self, 'default_value' ) else None
+ return self.default_value
# --- end of get_default_value (...) ---
@@ -146,7 +142,7 @@ class DescriptionField:
def get_flags ( self ):
"""Returns the flags of this DescriptionField or an empty list (=no flags)."""
- return self.flags if hasattr ( self, 'flags' ) else []
+ return self.flags
# --- end of get_flags (...) ---
@@ -155,7 +151,7 @@ class DescriptionField:
"""Returns the allowed values of this DescriptionField or an empty list,
which should be interpreted as 'no value restriction'.
"""
- return self.allowed_values if hasattr ( self, 'allowed_values' ) else []
+ return self.allowed_values
# --- end of get_allowed_values (...) ---
@@ -181,9 +177,6 @@ class DescriptionField:
if not field_identifier:
# bad identifier
return False
- elif not hasattr ( self, aliases ):
- # no aliases
- return False
if 'withcase' in self.aliases:
if field_identifier in self.aliases ['withcase']:
@@ -194,6 +187,8 @@ class DescriptionField:
if field_id_lower in self.aliases ['nocase']:
return True
+ return False
+
# --- end of matches_alias (...) ---
@@ -203,9 +198,6 @@ class DescriptionField:
arguments:
* flag --
"""
- if not hasattr ( self, 'flags' ):
- return False
-
return bool ( flag.lower() in self.flags )
def value_allowed ( self, value, nocase=True ):
@@ -215,18 +207,17 @@ class DescriptionField:
* value -- value to check
* nocase -- if True (the default): be case insensitive
"""
- allowed_values = self.get_allowed_values ()
- if not allowed_values:
+ if not self.allowed_values:
return True
elif nocase:
lowval = value.lower()
- for allowed in allowed_values:
+ for allowed in self.allowed_values:
if allowed.lower() == lowval:
return True
else:
- return bool ( value in allowed_values )
+ return bool ( value in self.allowed_values )
return False
@@ -315,22 +306,21 @@ class DescriptionFields:
flagmap = dict ()
optionmap = dict (
defaults = dict (),
- allowed_values = set ()
+ allowed_values = list ()
)
for field_name in self.fields.keys():
-
- d = self.fields [field_name].get_default_value()
+ d = self.fields [field_name].default_value
if not d is None:
optionmap ['defaults'] [field_name] = d
- if self.fields [field_name].get_allowed_values():
- optionmap ['allowed_values'].add ( field_name )
+ if self.fields [field_name].allowed_values:
+ optionmap ['allowed_values'].append ( field_name )
- for flag in self.fields [field_name].get_flags():
+ for flag in self.fields [field_name].flags:
if not flag in flagmap:
- flagmap [flag] = set ()
- flagmap [flag].add ( field_name )
+ flagmap [flag] = list ()
+ flagmap [flag].append ( field_name )
self._fields_by_flag = flagmap
self._fields_by_option = optionmap
next reply other threads:[~2012-06-06 19:53 UTC|newest]
Thread overview: 159+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-06 19:52 André Erdmann [this message]
-- strict thread matches above, loose matches on Subject: below --
2015-01-26 17:41 [gentoo-commits] proj/R_overlay:master commit in: roverlay/ André Erdmann
2015-01-26 17:41 André Erdmann
2014-07-18 16:20 André Erdmann
2014-07-18 2:50 [gentoo-commits] proj/R_overlay:wip/addition_control " André Erdmann
2014-07-18 16:20 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2014-07-18 2:28 [gentoo-commits] proj/R_overlay:wip/addition_control " André Erdmann
2014-07-18 16:20 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2014-07-16 15:14 André Erdmann
2014-06-05 22:09 André Erdmann
2014-04-01 16:38 André Erdmann
2014-02-16 16:30 André Erdmann
2014-02-15 19:49 André Erdmann
2014-02-15 19:49 André Erdmann
2014-01-25 18:14 André Erdmann
2013-12-11 18:40 André Erdmann
2013-12-11 18:40 André Erdmann
2013-09-23 15:30 André Erdmann
2013-09-20 15:57 André Erdmann
2013-09-19 15:00 André Erdmann
2013-09-17 16:40 André Erdmann
2013-09-17 16:40 André Erdmann
2013-09-17 16:40 André Erdmann
2013-09-17 16:40 André Erdmann
2013-09-16 13:43 André Erdmann
2013-09-13 15:10 André Erdmann
2013-09-12 16:36 André Erdmann
2013-09-12 16:36 André Erdmann
2013-09-12 16:36 André Erdmann
2013-09-11 11:14 André Erdmann
2013-09-11 10:19 André Erdmann
2013-09-10 14:40 André Erdmann
2013-09-10 14:40 André Erdmann
2013-09-10 14:40 André Erdmann
2013-09-10 14:40 André Erdmann
2013-09-06 17:27 André Erdmann
2013-09-06 17:27 André Erdmann
2013-09-03 15:50 André Erdmann
2013-09-02 12:27 André Erdmann
2013-09-02 8:44 André Erdmann
2013-08-30 14:49 André Erdmann
2013-08-30 14:49 André Erdmann
2013-08-29 12:36 André Erdmann
2013-08-29 12:36 André Erdmann
2013-08-28 15:54 André Erdmann
2013-08-27 15:39 André Erdmann
2013-08-23 13:52 André Erdmann
2013-08-23 13:52 André Erdmann
2013-08-23 13:52 André Erdmann
2013-08-19 15:42 André Erdmann
2013-08-16 14:05 André Erdmann
2013-08-16 11:02 André Erdmann
2013-08-16 10:43 André Erdmann
2013-08-16 10:43 André Erdmann
2013-08-14 14:56 André Erdmann
2013-08-14 14:56 André Erdmann
2013-08-13 8:56 André Erdmann
2013-08-13 8:56 André Erdmann
2013-08-13 8:56 André Erdmann
2013-08-12 8:28 André Erdmann
2013-08-12 8:18 André Erdmann
2013-08-07 16:10 André Erdmann
2013-08-02 14:30 André Erdmann
2013-08-02 10:34 André Erdmann
2013-08-02 10:34 André Erdmann
2013-08-01 12:44 André Erdmann
2013-08-01 12:44 André Erdmann
2013-07-29 14:56 André Erdmann
2013-07-29 8:55 André Erdmann
2013-07-26 13:02 André Erdmann
2013-07-23 7:51 André Erdmann
2013-07-23 7:51 André Erdmann
2013-07-19 18:00 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-07-23 7:51 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-07-17 18:05 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-07-17 18:05 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-07-15 22:31 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-07-16 16:36 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-07-12 13:57 André Erdmann
2013-06-22 15:24 André Erdmann
2013-06-22 15:24 André Erdmann
2013-06-22 15:24 André Erdmann
2013-06-22 15:24 André Erdmann
2013-06-19 18:58 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-06-22 15:24 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-06-19 18:58 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-06-19 18:59 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-06-13 16:34 André Erdmann
2013-06-05 18:08 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-06-13 16:34 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-06-05 18:08 [gentoo-commits] proj/R_overlay:gsoc13/next " André Erdmann
2013-06-13 16:34 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2013-06-04 21:06 André Erdmann
2013-04-25 16:44 André Erdmann
2013-04-25 16:44 André Erdmann
2013-03-05 11:27 André Erdmann
2013-02-09 20:45 André Erdmann
2013-02-05 17:48 André Erdmann
2013-02-05 17:48 André Erdmann
2013-01-30 20:16 André Erdmann
2013-01-30 20:16 André Erdmann
2013-01-28 23:54 André Erdmann
2013-01-28 23:54 André Erdmann
2013-01-28 23:54 André Erdmann
2012-10-02 10:04 André Erdmann
2012-08-20 11:16 André Erdmann
2012-08-13 18:07 André Erdmann
2012-08-09 9:26 André Erdmann
2012-08-08 23:46 André Erdmann
2012-08-08 23:46 André Erdmann
2012-08-07 8:50 André Erdmann
2012-08-02 15:14 André Erdmann
2012-08-01 7:25 André Erdmann
2012-07-31 17:51 André Erdmann
2012-07-30 8:52 André Erdmann
2012-07-30 8:52 André Erdmann
2012-07-24 16:59 [gentoo-commits] proj/R_overlay:overlay_wip " André Erdmann
2012-07-30 8:52 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2012-07-18 16:49 [gentoo-commits] proj/R_overlay:overlay_wip " André Erdmann
2012-07-30 8:52 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2012-07-16 16:15 André Erdmann
2012-07-16 16:15 André Erdmann
2012-07-16 16:15 [gentoo-commits] proj/R_overlay:depres_wip " André Erdmann
2012-07-16 16:15 ` [gentoo-commits] proj/R_overlay:master " André Erdmann
2012-07-10 17:43 André Erdmann
2012-07-09 17:19 André Erdmann
2012-07-04 18:21 André Erdmann
2012-07-04 18:21 André Erdmann
2012-07-03 17:48 André Erdmann
2012-06-28 13:29 André Erdmann
2012-06-26 15:42 André Erdmann
2012-06-25 18:19 André Erdmann
2012-06-21 16:55 André Erdmann
2012-06-20 19:03 André Erdmann
2012-06-20 19:03 André Erdmann
2012-06-18 16:27 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-15 20:34 André Erdmann
2012-06-12 17:17 André Erdmann
2012-06-06 19:52 André Erdmann
2012-06-06 19:52 André Erdmann
2012-06-05 17:30 André Erdmann
2012-06-04 19:07 André Erdmann
2012-06-04 19:07 André Erdmann
2012-06-04 15:43 André Erdmann
2012-06-01 16:19 André Erdmann
2012-06-01 16:19 André Erdmann
2012-06-01 15:46 André Erdmann
2012-05-31 18:24 André Erdmann
2012-05-30 20:15 André Erdmann
2012-05-30 19:36 André Erdmann
2012-05-30 19:36 André Erdmann
2012-05-30 16:09 André Erdmann
2012-05-30 16:09 André Erdmann
2012-05-30 16:09 André Erdmann
2012-05-30 16:09 André Erdmann
2012-05-30 10:58 André Erdmann
2012-05-30 10:58 André Erdmann
2012-05-30 10:58 André Erdmann
2012-05-30 10:58 André Erdmann
2012-05-29 17:09 André Erdmann
2012-05-29 17:09 André Erdmann
2012-05-29 17:09 André Erdmann
2012-05-29 17:09 André Erdmann
2012-05-29 17:09 André Erdmann
2012-05-26 13:14 André Erdmann
2012-05-26 13:14 André Erdmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1339012078.1f51ac4acac24633cba9d7a531c7165af7acd09d.dywi@gentoo \
--to=dywi@mailerd.de \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox