Re: [gentoo-portage-dev] [PATCH v3 1/2] dblink: case insensitive support for bug #524236

[Brian Dolbec <dolsen@gentoo.org>]

On Mon, 17 Nov 2014 12:29:37 -0800
Zac Medico <zmedico@gentoo.org> wrote:

> This adds a dblink._contents attribute with methods that provide
> an interface for contents operations with "implicit" case handling.
> The new methods are implemented in a separate
> ContentsCaseSensitivityManager class, in order to avoid adding more
> bloat to vartree.py.
> 
> X-Gentoo-Bug: 524236
> X-Gentoo-Url: https://bugs.gentoo.org/show_bug.cgi?id=524236
> ---
...
> +class ContentsCaseSensitivityManager(object):
> +	"""
> +	Implicitly handles case transformations that are needed for
> +	case-insensitive support.
> +	"""
> +
> +	def __init__(self, db):
> +		"""
> +		@param db: A dblink instance
> +		@type db: vartree.dblink
> +		"""


Thank you :)  looks much better now ;)

Both 1/2, 2/2 look good now.  Clear to merge :)



> +		self.getcontents = db.getcontents
> +
> +		if "case-insensitive-fs" in db.settings.features:
> +			self.unmap_key =
> self._unmap_key_case_insensitive
> +			self.contains =
> self._contains_case_insensitive
> +			self.keys = self._keys_case_insensitive
> +
> +		self._contents_insensitive = None
> +		self._reverse_key_map = None
> +
> +	def clear_cache(self):
> +		"""
> +		Clear all cached contents data.
> +		"""
> +		self._contents_insensitive = None
> +		self._reverse_key_map = None
> +
> +	def keys(self):
> +		"""
> +		Iterate over all contents keys, which are
> transformed to
> +		lowercase when appropriate, for use in
> case-insensitive
> +		comparisons.
> +		@rtype: iterator
> +		@return: An iterator over all the contents keys
> +		"""
> +		return iter(self.getcontents())
> +
> +	def contains(self, key):
> +		"""
> +		Check if the given key is contained in the contents,
> using
> +		case-insensitive comparison when appropriate.
> +		@param key: A filesystem path (including ROOT and
> EPREFIX)
> +		@type key: str
> +		@rtype: bool
> +		@return: True if the given key is contained in the
> contents,
> +			False otherwise
> +		"""
> +		return key in self.getcontents()
> +
> +	def unmap_key(self, key):
> +		"""
> +		Map a key (from the keys method) back to its
> case-preserved
> +		form.
> +		@param key: A filesystem path (including ROOT and
> EPREFIX)
> +		@type key: str
> +		@rtype: str
> +		@return: The case-preserved form of key
> +		"""
> +		return key
> +
> +	def _case_insensitive_init(self):
> +		"""
> +		Initialize data structures for case-insensitive
> support.
> +		"""
> +		self._contents_insensitive = dict(
> +			(k.lower(), v) for k, v in
> self.getcontents().items())
> +		self._reverse_key_map = dict(
> +			(k.lower(), k) for k in self.getcontents())
> +
> +	def _keys_case_insensitive(self):
> +		if self._contents_insensitive is None:
> +			self._case_insensitive_init()
> +		return iter(self._contents_insensitive)
> +
> +	_keys_case_insensitive.__doc__ = keys.__doc__
> +
> +	def _contains_case_insensitive(self, key):
> +		if self._contents_insensitive is None:
> +			self._case_insensitive_init()
> +		return key.lower() in self._contents_insensitive
> +
> +	_contains_case_insensitive.__doc__ = contains.__doc__
> +
> +	def _unmap_key_case_insensitive(self, key):
> +		if self._reverse_key_map is None:
> +			self._case_insensitive_init()
> +		return self._reverse_key_map[key]
> +
> +	_unmap_key_case_insensitive.__doc__ = unmap_key.__doc__
> diff --git a/pym/portage/dbapi/vartree.py
> b/pym/portage/dbapi/vartree.py index 8b06f4c..81059b1 100644
> --- a/pym/portage/dbapi/vartree.py
> +++ b/pym/portage/dbapi/vartree.py
> @@ -69,6 +69,7 @@ from _emerge.EbuildPhase import EbuildPhase
>  from _emerge.emergelog import emergelog
>  from _emerge.MiscFunctionsProcess import MiscFunctionsProcess
>  from _emerge.SpawnProcess import SpawnProcess
> +from ._ContentsCaseSensitivityManager import
> ContentsCaseSensitivityManager 
>  import errno
>  import fnmatch
> @@ -1525,6 +1526,7 @@ class dblink(object):
>  		# When necessary, this attribute is modified for
>  		# compliance with RESTRICT=preserve-libs.
>  		self._preserve_libs = "preserve-libs" in
> mysettings.features
> +		self._contents = ContentsCaseSensitivityManager(self)
>  
>  	def __hash__(self):
>  		return hash(self._hash_key)
> @@ -1612,6 +1614,7 @@ class dblink(object):
>  		self.contentscache = None
>  		self._contents_inodes = None
>  		self._contents_basenames = None
> +		self._contents.clear_cache()
>  
>  	def getcontents(self):
>  		"""



-- 
Brian Dolbec <dolsen>