[Tickets #4882] Re: IMSP backend - speedup browse with many groups

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://dev.horde.org/horde/whups/ticket/?id=4882
-----------------------------------------------------------------------
 Ticket             | 4882
 Updated By         | Michael Rubinsky <mrubinsk at horde.org>
 Summary            | IMSP backend - speedup browse with many groups
 Queue              | Turba
 Version            | HEAD
 Type               | Enhancement
 State              | Assigned
 Priority           | 1. Low
 Owners             | Michael Rubinsky
-----------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2007-01-12 21:15) wrote:

I can't get this patch to apply cleanly to HEAD.   Can you please create a
new patch against a freshly updated copy of imsp.php?

Also, is there a specific reason you choose to use array_push() to add a
single value to the end of the $id  array?  Using $id[] = $id[$i] would be
slightly more efficient, as it avoids the overhead of a function call.

Just curious, did you do any benchmarking to see what the improvement in
performance was?  

Thanks!