diff options
| author | Jason Madden <jamadden@gmail.com> | 2020-01-23 11:38:41 -0600 |
|---|---|---|
| committer | Jason Madden <jamadden@gmail.com> | 2020-01-23 11:38:41 -0600 |
| commit | 72dc05b90fb26385f0ad100376e0875439825da8 (patch) | |
| tree | 6e8a8ddb202f76cb1dad8b90a1ee077a5f0bf752 /src/zope/interface | |
| parent | cacd85b78d530336136cac40c42019b8b6946eb0 (diff) | |
| download | zope-interface-slots.tar.gz | |
Avoid allocating space for tagged values unless they're used. This saves another few percent.slots
Diffstat (limited to 'src/zope/interface')
| -rw-r--r-- | src/zope/interface/interface.py | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/zope/interface/interface.py b/src/zope/interface/interface.py index c374cef..f501077 100644 --- a/src/zope/interface/interface.py +++ b/src/zope/interface/interface.py @@ -67,7 +67,9 @@ class Element(object): self.__name__ = __name__ self.__doc__ = __doc__ - self.__tagged_values = {} + # Tagged values are rare, especially on methods or attributes. + # Deferring the allocation can save substantial memory. + self.__tagged_values = None def getName(self): """ Returns the name of the object. """ @@ -79,18 +81,22 @@ class Element(object): def getTaggedValue(self, tag): """ Returns the value associated with 'tag'. """ + if not self.__tagged_values: + raise KeyError(tag) return self.__tagged_values[tag] def queryTaggedValue(self, tag, default=None): """ Returns the value associated with 'tag'. """ - return self.__tagged_values.get(tag, default) + return self.__tagged_values.get(tag, default) if self.__tagged_values else default def getTaggedValueTags(self): """ Returns a list of all tags. """ - return self.__tagged_values.keys() + return self.__tagged_values.keys() if self.__tagged_values else () def setTaggedValue(self, tag, value): """ Associates 'value' with 'key'. """ + if self.__tagged_values is None: + self.__tagged_values = {} self.__tagged_values[tag] = value |
