(not reading, please use seth@sspitzer.org instead) Reporter
	Description • 24 years ago

as long as TreeBoxObject::BeginBatch() / TreeBoxObject::EndBatch() does the
right thing, we can clean up some of the code in XULSortService::DoSort().

I'll attach a patch.

	(not reading, please use seth@sspitzer.org instead) Reporter
	Comment 2 • 24 years ago

in that patch, I fixed some warnings, switched over to the tree batching api,
and added some timing code.

while working on mailnews sort performance, I found that timing code very
useful.

accepting, adding dependency on a tree batching bug #66583

	(not reading, please use seth@sspitzer.org instead) Reporter
	Comment 3 • 24 years ago

well, I have no plans on doing this now.

in mailnews, we are doing the sorting in our view.  giving this to waterson.

feel free to mark this wont fix.

	Chase Tingley
	Comment 4 • 23 years ago

I'm going to try to fold this in to my work on bug 105783.

	Chase Tingley
	Comment 5 • 23 years ago

I had this patch in my tree for a bit, but things were acting strangely -- I was
getting crashes when trying to scroll after doing a quicksort on a tree.  I'll
try to investigate at some point.

	Joe Hewitt (gone)
	Comment 6 • 23 years ago

tree batching? we don't got no stinking tree batching

	benc
	Comment 7 • 21 years ago

tever is not RDF QA anymore