getChildValue() on GeoRSS format is slow

[edgemaster]

The getChildValue function on Format.GeoRSS seriously slows down parsing of large documents (~100 features) The firebug profiler says that each call is averaging 5.57ms, with max of 500ms, totalling 2785ms for 500 calls.

On a hunch, I patched the function to not use an exception handler, this resulted in a considerable speedup, avg 0.406ms, max 0.908ms, total 118.442ms for the same 500 calls.

Attached is a testcase and patch to current trunk, georss tests pass.

[edgemaster]

Patch to Format.GeoRSS

[edgemaster]

Testcase (for fx2 & fb1.2)

[edgemaster]

georss for testcase

[edgemaster]

Testcase is also online at  http://ge.pythonmoo.co.uk/maps/georssGCVtest.htm

[elemoine]

Patch looks good to me. I'll go ahead and commit it if there's no push back in the coming days. Thanks edgemaster.

[crschmidt]

+1

[edgemaster]

(Just accepting as my ticket for later reference)

[elemoine]

Except one unrelated error in Lang.html, all unit tests pass on FF2.

[elemoine]

(In [7468]) getChildValue() on GeoRSS format is slow, p=edgemaster, r=me,crschmidt (closes #1600)