* Initialize dbapi._categories = None so that we can compare

  with None instead of using hasattr.
* Remove unwanted self._categories initialization from the
  portdbapi constructor.
(trunk r9092)

svn path=/main/branches/2.1.2/; revision=9093

1 files changed, 8 insertions, 8 deletions

diff --git a/pym/portage.py b/pym/portage.py
index cbf3496b0..4524c049b 100644
--- a/pym/portage.py
+++ b/pym/portage.py
@@ -5811,6 +5811,7 @@ class portagetree:
 class dbapi:
 	_category_re = re.compile(r'^\w[-.+\w]*$')
 	_pkg_dir_name_re = re.compile(r'^\w[-+\w]*$')
+	_categories = None
 	def __init__(self):
 		pass
 
@@ -5821,7 +5822,7 @@ class dbapi:
 		can delete the self._categories attribute in cases when the cached
 		categories become invalid and need to be regenerated.
 		"""
-		if hasattr(self, "_categories"):
+		if self._categories is not None:
 			return self._categories
 		categories = set()
 		cat_pattern = re.compile(r'(.*)/.*')
@@ -5947,8 +5948,8 @@ class fakedbapi(dbapi):
 		self._match_cache = {}
 
 	def _clear_cache(self):
-		if hasattr(self, "_categories"):
-			del self._categories
+		if self._categories is not None:
+			self._categories = None
 		if self._match_cache:
 			self._match_cache = {}
 
@@ -6774,7 +6775,6 @@ class portdbapi(dbapi):
 		else:
 			global settings
 			self.mysettings = config(clone=settings)
-		self._categories = set(self.mysettings.categories)
 		# This is strictly for use in aux_get() doebuild calls when metadata
 		# is generated by the depend phase.  It's safest to use a clone for
 		# this purpose because doebuild makes many changes to the config
@@ -8167,8 +8167,8 @@ class dblink:
 		The caller must ensure that lockdb() and unlockdb() are called
 		before and after this method.
 		"""
-		if hasattr(self.vartree.dbapi, "_categories"):
-			del self.vartree.dbapi._categories
+		if self.vartree.dbapi._categories is not None:
+			self.vartree.dbapi._categories = None
 		# When others_in_slot is supplied, the security check has already been
 		# done for this slot, so it shouldn't be repeated until the next
 		# replacement or unmerge operation.
@@ -9396,8 +9396,8 @@ class dblink:
 		we won't be able to later if they get unmerged (happens
 		when namespace changes).
 		"""
-		if hasattr(self.vartree.dbapi, "_categories"):
-			del self.vartree.dbapi._categories
+		if self.vartree.dbapi._categories is not None:
+			self.vartree.dbapi._categories = None
 		if self.myroot == "/" and \
 			"sys-apps" == self.cat and \
 			"portage" == pkgsplit(self.pkg)[0] and \