Closed Bug 455536 Opened 16 years ago Closed 6 years ago

outparamdel remaining nsNodeInfoManager::GetNodeInfo

Categories

(Developer Infrastructure :: Source Code Analysis, defect)

Product:
Developer Infrastructure
Component:
Source Code Analysis
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: taras.mozilla, Unassigned)

References

Details

Attachments

(2 files)

remaining GetNodeInfo outparamdelling
4.84 KB, patch
Details | Diff | Splinter Review
s/NS_ERROR_FAILURE/NS_ERROR_OOM/ in previous patch
24.31 KB, patch
jst
: review+
jst
: superreview+
Details | Diff | Splinter Review 
+++ This bug was initially created as a clone of Bug #450777 +++

patch coming soon
Blocks: 266048
Summary: outparamdel nsNodeInfoManager::GetNodeInfo → outparamdel remaining nsNodeInfoManager::GetNodeInfo
Depends on: 455540
bz suggested that I make nsNodeInfoManager::GetNodeInfo to only fail with NS_ERROR_OUT_OF_MEMORY(instead of NS_ERROR_FAILURE), so I'll also include a followup patch to bug 450777 to change those error codes.
bz, will get rid of the other overload.
Attachment #338914 - Flags: review?(jst)
Blocks: static_analyses
No longer blocks: 266048
This switches error code in bug 450777 to be NS_ERROR_OUT_OF_MEMORY
Attachment #339287 - Flags: review?(jst)
Attachment #339287 - Flags: superreview+
Attachment #339287 - Flags: review?(jst)
Attachment #339287 - Flags: review+
checked in S/FAILURE/OOM/ in 514ea093a63f
Attachment #338914 - Flags: review?(jst)
Product: Core → Firefox Build System
Don't think it is still relevant (and probably not in the right component) closing.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: