Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Content-Class: urn:content-classes:message
Message-ID: <3FF644DB.6030004@steitz.com>
Date: Fri, 02 Jan 2004 21:28:11 -0700
From: "Phil Steitz" <phil@steitz.com>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20030225
MIME-Version: 1.0
To: "Jakarta Commons Developers List" <commons-dev@jakarta.apache.org>
Subject: Re: [collections] CaseInsensitiveHashMap
References: <7907946.1073048659641.JavaMail.root@127.0.0.1>
 <3FF59F2C.8060407@steitz.com> <004301c3d184$07bb27a0$24638051@oemcomputer>
In-Reply-To: <004301c3d184$07bb27a0$24638051@oemcomputer>
Content-Type: text/plain;
	format=flowed;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit

Stephen Colebourne wrote:
> From: "Phil Steitz" <phil@steitz.com>
> 
>>>>2. I made no attempt at allowing the locale to be set or otherwise
>>>>changing the default behavior of toLower() in convert().
>>>
>>>I'm wondering how your implementation works. I was expecting the
> 
> isEqualKey() method to be overridden, rather than converting the key on add.
> (Note that the same code also has to go in a MapEntry)
> 
>>Could be I was being too lazy.  Since toLower() is "idempotent" (calling
>>it repeatedly has same effect as calling it just once) it "works" to just
>>override convert(), since in AbstractHashedMap, get, put, remove,
>>containsKey, all start by converting the key (nice!).  All I did was to
>>extend HashedMap and override convert to return
>>key.toString().toLowerCase().  The effect is that you actually store all
>>lower case strings as keys (or null, once I fix this). The behavior should
>>be the same as the impl that David Graham posted with PR #24537 (unless I
>>am missing something -- quite possible).
> 
> This impl will work, but I would prefer to see the case of the key
> maintained, just the comparison to be case insensitive.

Why exactly?  Would you expect equals to distinguish 
CaseInsensitiveHashMaps that differ only on key case?

Phil

> 
> Stephen
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org