[OpenLayers-Dev] Performance problems in changeset [2867]

[Erik Uzureau]

Hi James,

Thanks very much for this patch. We weren't even taking into
consideration speed when we were writing these patches -- only memory.
How awesome to see how quickly the community picks up on things like
this.

Do we have an ICLA / CCLA from you? This is a very elegant patch and I
would love to include it in trunk, but must first have the proper
authorizations.

http://trac.openlayers.org/wiki/HowToContribute

Erik

On 3/23/07, James Stembridge <James.Stembridge at sandfordtechnology.com> wrote:
> Hi,
>
> While improving things in relation to memory usage changeset [2867]
> seems to have introduced a pretty unacceptable decline in performance.
> I'm finding that removing ~400 markers from a layer takes ~4s, with the
> vast proportion of the time spent in Event.stopObserving.
>
> I have attached (I can't seem to do this in trac) a patch that improves
> the efficiency of the observer cache with respect to removals, while not
> perfect this drops the time taken to remove ~400 markers to a more
> respectable 500ms.
>
> Regards,
> James.
>
>
> --
>
> DISCLAIMER: Important - This e-mail and the information that it contains
> may be confidential, legally privileged and protected by law. Access by
> the intended recipient only is authorised. Any liability (in negligence
> or otherwise) arising from any third party acting, or refraining from
> acting, on any information contained in this e-mail is hereby excluded.
> If you are not the intended recipient, please notify the sender
> immediately and do not disclose the contents to any other person, use it
> for any purpose, or store or copy the information in any medium.
> Copyright in this e-mail and attachments created by us belongs to
> Sandford Technology. The author also asserts the right to be identified
> as such and objects to any misuse.
>
> Company no: 04397593 Reg office: 1 City Square, Leeds, LS1 2DP, United
> Kingdom VAT no: 789 6572 50
>
> _______________________________________________
> Dev mailing list
> Dev at openlayers.org
> http://openlayers.org/mailman/listinfo/dev
>
>
>