[josm-dev] Changeset 2427

2009-11-10 Thread Dave Hansen
This changeset: http://josm.openstreetmap.de/changeset/2427/ removes the way_bbox_cache. Unfortunately, this just *KILLS* performance. It makes some of my way search tests 10-20x slower. We use getBBox() really, really often and the CPU we use repeatedly recreating them ends up being t

Re: [josm-dev] Changeset 2427

2009-11-10 Thread Jiri Klement
It's not possible to get it easily back as it was, but Way.getBBox() should do the caching now. When I get time I'll add events to Dataset so that Way will know when it's nodes were changed and update it's bbox. On Tue, Nov 10, 2009 at 6:33 PM, Dave Hansen wrote: > This changeset: > >        http

Re: [josm-dev] Changeset 2427

2009-11-10 Thread Dave Hansen
On Tue, 2009-11-10 at 18:37 +0100, Jiri Klement wrote: > It's not possible to get it easily back as it was, but Way.getBBox() > should do the caching now. I'm not sure it's implemented yet: @Override public BBox getBBox() { // TODO Precalculate way bbox (and update it every time n

Re: [josm-dev] Changeset 2427

2009-11-11 Thread Dave Hansen
On Tue, 2009-11-10 at 18:37 +0100, Jiri Klement wrote: > It's not possible to get it easily back as it was, but Way.getBBox() > should do the caching now. When I get time I'll add events to Dataset > so that Way will know when it's nodes were changed and update it's > bbox. Just saw the commit. T