vijayakumar palaniappan | 22 Jun 2005 12:11
Picon
Favicon

AbstractElement perf changes

getDeclaredNamespaces() seems to create a new TreeSet
and a new Comaprator object. Also it does a copy of
all the elements in to the new Treeset.

getNamespaceByURI() calls getDeclaredNamespaces().
Since getNamespaceByURI() gets called frequently and
it does not modify _NSS, it is better to access it
directly rather than through getDeclaredNamespaces().

listElements() uses a new NSI compartor. Probably it
can reuse the comparator object defined in NSI

Attached are the diffs.

Thanks,
Vijay

		
____________________________________________________ 
Yahoo! Sports 
Rekindle the Rivalries. Sign up for Fantasy Football 
http://football.fantasysports.yahoo.com
Attachment (AbstratElement.diff1): application/octet-stream, 516 bytes
_______________________________________________
jso-devel mailing list
jso-devel <at> jabberstudio.org
http://mail.jabber.org/mailman/listinfo/jso-devel
vijayakumar palaniappan | 23 Jun 2005 05:39
Picon
Favicon

Re: AbstractElement perf changes

The changes in the atrtachment are backwards.
Attached the new diffs

--- vijayakumar palaniappan <vijayakumarpl <at> yahoo.com>
wrote:

> getDeclaredNamespaces() seems to create a new
> TreeSet
> and a new Comaprator object. Also it does a copy of
> all the elements in to the new Treeset.
> 
> getNamespaceByURI() calls getDeclaredNamespaces().
> Since getNamespaceByURI() gets called frequently and
> it does not modify _NSS, it is better to access it
> directly rather than through
> getDeclaredNamespaces().
> 
> listElements() uses a new NSI compartor. Probably it
> can reuse the comparator object defined in NSI
> 
> Attached are the diffs.
> 
> Thanks,
> Vijay
> 
> 
> 		
> ____________________________________________________
> 
> Yahoo! Sports 
(Continue reading)


Gmane