« Return to Thread: qx.ui.table.model.Simple.setSortMethods()

Re: qx.ui.table.model.Simple.setSortMethods()

by Derrell Lipman :: Rate this Message:

Reply to Author | View in Thread

On Wed, Jul 8, 2009 at 10:21 AM, dmbaggett <dave@...> wrote:


The argument given in the source for having two methods is "performance
reasons". But swapping parameters obviously causes no performance
degradation.

Actually, I don't believe that's true. It imposes an extra function call for each two items that are compared. Depending upon the sort algorithm that's implemented in the native Array.sort() and how out-of-order the items are to begin with, this means a likely minimum number of extra function calls is the number of rows in the table, and the actual number could be much, much greater.

For the new (to-be-named) enhanced table model which I recently submitted as
an enhancement, this is a trivial change.

The only question is whether we want to do anything to avoid breaking
existing code as this will constitute an API change.

See my proposed implementation that handles this and associated comments in bug #2553:
  http://bugzilla.qooxdoo.org/show_bug.cgi?id=2553

Derrell


------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time,
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@...
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

 « Return to Thread: qx.ui.table.model.Simple.setSortMethods()