no the api is just opened up to be able to create your own map impl I can change it to be ICache. But the localizer currently just needs a Map<String,String> thats a pretty common thing to have.
But i agree wit that contains call. thats just double up because if it is we have too look it up again. I will remove it. On Mon, May 12, 2008 at 11:39 AM, Jan Kriesten <[EMAIL PROTECTED]> wrote: > > hi johan, > > why? > > we just have a protected factory method now for a cache implementation > > that > > must be a map. > > > > if you open up the api to change the implementation of the cache - there > are other implementations than map... why limit it to map? (actually, I > don't really understand the cache.containsKey(cacheKey) at all since it > could just be checked for null when getting the value)... > > > best regards, --- jan. > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > >
