menu_get_item() can be optimized a little in the case where the item is in the cache (as a side note - wth are we using the cache system for this?).

CommentFileSizeAuthor
#2 964822-opt-menu-get-item-1.patch972 bytespwolanin

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Background: #643984: Cache menu_get_item()

(Don't look at me.)

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new 964822-opt-menu-get-item-1.patch972 bytes

just moves 2 lines inside the conditional.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community

heh. Only with this patch, the db cache starts to make sense. :)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.