| Summary | Don't use array_push |
| Queue | Horde Base |
| Queue Version | HEAD |
| Type | Enhancement |
| State | Resolved |
| Priority | 1. Low |
| Owners | |
| Requester | selsky (at) columbia (dot) edu |
| Created | 4/7/05 (7753 days ago) |
| Due | |
| Updated | 4/8/05 (7752 days ago) |
| Assigned | |
| Resolved | 4/8/05 (7752 days ago) |
| Milestone | |
| Patch | No |
State ⇒ Resolved
Note: If you use array_push() to add one element to the array it's
better to use $array[] = because in that way there is no overhead of
calling a function.
I'm not positive that it's applicable here since we're calling
array(), but array() isn't a real function.
I only submitted this patch since it took 5 seconds and reduced a few
lines of code that I had added.
performance problems for you? Just thinking that optimization efforts
could be better focused elsewhere...
State ⇒ Feedback
this from your own benchmarks?
State ⇒ New
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Don't use array_push
Queue ⇒ Horde Base
New Attachment: remove_prefs.patch
that function can be slow.