good work Michael.
I am wondering too why they used simple "Lists" ... maybe it was just a 
thing that they did in the beginning?
One thought I have is about serialization or web services.. but all of 
that stuff is internally, so I can't really think of a connection.

anyway, thanks again
stefan


Am 18.09.2011 08:57, schrieb Michaël Michaud:
> Hi,
>
> I have made important modifications in EditTransaction class in order to
> improve performance.
> 1 - I have replaced most Lists by Sets and Maps avoiding the
> List.contains() method which is far too slow.
> 2 - I have removed geometry cloning in createFeature and deleteFeature
> methods, avoiding time and memory consuming operations.
>
> I cannot say I'm 100% sure cloning was useless in every case, but I made
> a lot of tests to checks it does not beark anythink. Please, report any
> suspect behaviour (from next NB).
>
> You'll see the benefits if you compare the following operation which
> uses a undoable transaction :
> Explode a multigeometry having 180000 small polygons
> Before
> 2mn 50s - memory increase from 160Mb to 425Mb
> After
> 10 s - memory increase from 160Mb to 260Mb
>
> Michaël
>
>
> ------------------------------------------------------------------------------
> BlackBerry® DevCon Americas, Oct. 18-20, San Francisco, CA
> http://p.sf.net/sfu/rim-devcon-copy2
> _______________________________________________
> Jump-pilot-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>


------------------------------------------------------------------------------
BlackBerry® DevCon Americas, Oct. 18-20, San Francisco, CA
Learn about the latest advances in developing for the 
BlackBerry® mobile platform with sessions, labs & more.
See new tools and technologies. Register for BlackBerry® DevCon today!
http://p.sf.net/sfu/rim-devcon-copy1 
_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to