optimization for 'rare' oom check

RESOLVED FIXED

Status

()

--
trivial
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
370   if (!list) {
  373     list = new nsContentList(aDocument, aMatchAtom,
  374                              aMatchNameSpaceId, aRootContent);
  381   }
  383   NS_ENSURE_TRUE(list, NS_ERROR_OUT_OF_MEMORY);

optimization: swap 383 and 381
(Assignee)

Comment 2

16 years ago
Created attachment 126826 [details] [diff] [review]
reorder check so it is only reached when it makes sense
(Assignee)

Updated

16 years ago
Attachment #126826 - Flags: superreview?(bzbarsky)
Attachment #126826 - Flags: review?(bzbarsky)
(Assignee)

Updated

16 years ago
Attachment #126826 - Flags: superreview?(bzbarsky)
Attachment #126826 - Flags: superreview+
Attachment #126826 - Flags: review?(bzbarsky)
Attachment #126826 - Flags: review+
(Assignee)

Comment 3

16 years ago
checked in
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.