pivot-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Brown <gkbr...@mac.com>
Subject Re: [jira] Commented: (PIVOT-132) pivot/wtk/ApplicationContext.java:1201 pivot.wtk.ApplicationContext.resourceCache is or uses a map or set of URLs, which can be a performance hog
Date Fri, 16 Oct 2009 12:50:08 GMT
Yeah, after reading that article I agree. The article suggests using  
toString() rather than converting it to a URI - any pros/cons?


On Oct 16, 2009, at 8:33 AM, Todd Volkert wrote:

> http://daniel.gredler.net/2008/04/24/urlhashcode-considered-harmful/
> I disagree that we should wait on this if we have a fix in hand.   
> Tiggering
> DNS lookups upon hashCode()??  That's poor...
>
> -T
>
> On Fri, Oct 16, 2009 at 8:21 AM, Greg Brown <gkbrown@mac.com> wrote:
>
>> This seems like a reasonable fix, but I would personally prefer to  
>> leave
>> this issue assigned to the 1.5 release and address it then, if  
>> necessary (we
>> have not noticed any adverse effects from the current  
>> implementation so it
>> may not be worth changing).
>>
>>
>> On Oct 16, 2009, at 6:06 AM, Sandro Martini wrote:
>>
>> Hi Noel,
>>> thanks for the patch ... but what say others on this ?
>>>
>>> I (or you, no problem :-) ) could start applying it only to  
>>> ImageView,
>>> and see if all is good (no doubt it will be right), and then (maybe
>>> later) make the same for all other classes (but there are many),  
>>> ok ?
>>>
>>> Bye,
>>> Sandro
>>>
>>
>>


Mime
View raw message