diff options
| author | Zac Medico <zmedico@gentoo.org> | 2011-12-13 09:00:21 -0800 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2011-12-13 09:00:21 -0800 |
| commit | 70d7c7bfb20dec3fa66b0e6c1d9729ac87cab577 (patch) | |
| tree | a213980dca0e3fa6ec2183191f0f4b6d87da639d /pym | |
| parent | bffe72d567e2d4d4235ad45177fbec04da5260fa (diff) | |
| download | portage-70d7c7bfb20dec3fa66b0e6c1d9729ac87cab577.tar.gz portage-70d7c7bfb20dec3fa66b0e6c1d9729ac87cab577.tar.bz2 portage-70d7c7bfb20dec3fa66b0e6c1d9729ac87cab577.zip | |
config: use frozenset for _constant_keys lookup
Diffstat (limited to 'pym')
| -rw-r--r-- | pym/portage/package/ebuild/config.py | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/pym/portage/package/ebuild/config.py b/pym/portage/package/ebuild/config.py index 9a8486c39..f6166c65b 100644 --- a/pym/portage/package/ebuild/config.py +++ b/pym/portage/package/ebuild/config.py @@ -122,6 +122,9 @@ class config(object): virtuals ...etc you look in here. """ + _constant_keys = frozenset(['PORTAGE_BIN_PATH', 'PORTAGE_GID', + 'PORTAGE_PYM_PATH']) + _setcpv_aux_keys = ('DEFINED_PHASES', 'DEPEND', 'EAPI', 'INHERITED', 'IUSE', 'REQUIRED_USE', 'KEYWORDS', 'LICENSE', 'PDEPEND', 'PROPERTIES', 'PROVIDE', 'RDEPEND', 'SLOT', @@ -2126,14 +2129,16 @@ class config(object): def _getitem(self, mykey): - # These ones point to temporary values when - # portage plans to update itself. - if mykey == "PORTAGE_BIN_PATH": - return portage._bin_path - elif mykey == "PORTAGE_PYM_PATH": - return portage._pym_path - elif mykey == "PORTAGE_GID": - return _unicode_decode(str(portage_gid)) + if mykey in self._constant_keys: + # These two point to temporary values when + # portage plans to update itself. + if mykey == "PORTAGE_BIN_PATH": + return portage._bin_path + elif mykey == "PORTAGE_PYM_PATH": + return portage._pym_path + + elif mykey == "PORTAGE_GID": + return _unicode_decode(str(portage_gid)) for d in self.lookuplist: try: @@ -2186,9 +2191,7 @@ class config(object): def __iter__(self): keys = set() - keys.add("PORTAGE_BIN_PATH") - keys.add("PORTAGE_PYM_PATH") - keys.add("PORTAGE_GID") + keys.update(self._constant_keys) for d in self.lookuplist: keys.update(d) return iter(keys) |