Closed Bug 311886 Opened 20 years ago Closed 18 years ago

Change docs on NS_ENSURE_SUCCESS to mention warning, not asserting

Categories

(Developer Documentation Graveyard :: General, defect)

Product:
Developer Documentation Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Waldo, Unassigned)

References

(
URL
)

Details

Under "Use the nice macros" it says to use |NS_ENSURE_SUCCESS(rv, rv)| unless you don't want it to assert. This seems to be untrue according to <http://lxr.mozilla.org/mozilla/source/xpcom/glue/nsDebug.h#224> (see also line 239); NS_ENSURE_SUCCESS warns instead of asserts. The docs should reflect this.
Assignee: www-mozilla-org → nobody
QA Contact: danielwang → www-mozilla-org
Should migrate that to MDC and fix the text there.
Component: www.mozilla.org → Documentation Requests
Product: mozilla.org → Mozilla Developer Center
QA Contact: www-mozilla-org → doc-request
Version: other → unspecified
Not sure when it ended up at http://developer.mozilla.org/en/docs/Mozilla_Coding_Style_Guide#Use_the_nice_macros but there it is. Revision 63314.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Component: Documentation Requests → Documentation
Component: Documentation → General
Product: Mozilla Developer Network → Developer Documentation
You need to log in before you can comment on or make changes to this bug.