Ian, thanks for running the tests. Please attach results (both these and
tomorrow's) to the ticket so they don't get lost.

Thanks,

Derrell


On Wed, Jul 8, 2009 at 12:13 PM, Ian Horst <[email protected]> wrote:

> I run profiling using firebug profile on:
> var f1 = function (a, b) {return true;};
> var f2 = function (a, b) {return f1(b, a);};
>
> When I run 1000000 calls, I get stable performance penalty of:
> * 350ms on Intel I7 920 using Mozilla/5.0 (X11; U; Linux x86_64;
> en-GB; rv:1.9.1) Gecko/20090701 Gentoo Firefox/3.5
> * 1600ms on Intel 2.8Ghz HT using Firefox 3.0 on ArchLinux
>
> So far it seems that it makes sense to use f2() to sort small tables
> below 1000-1500 rows in worst case scenarios.
> Performance wise it's better to use predefined descending function.
>
> I'll run tests using patch made by Derrell tomorrow.
>
> ---
> Ian
>
> 2009/7/8 Derrell Lipman <[email protected]>:
> > On Wed, Jul 8, 2009 at 10:21 AM, dmbaggett <[email protected]> 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
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
> >
> >
>
>
> ------------------------------------------------------------------------------
> 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
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
>



-- 
The trick to education is to teach people in such a way that they don't
realize they're learning until it's too late.
 - Harold Eugene "Doc" Edgerton ("Poppa Flash"), 1903-1990, fabled MIT
professor and inventor of the stroboscopic flash
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to