The default bug view has changed. See this FAQ.

Use nullptr in a function that returns a pointer

RESOLVED FIXED in mozilla17

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: espindola, Assigned: espindola)

Tracking

unspecified
mozilla17
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 654475 [details] [diff] [review]
use nullptr
Attachment #654475 - Flags: review?(jonas)
Comment on attachment 654475 [details] [diff] [review]
use nullptr

Will this give the right error handling behavior?
Attachment #654475 - Flags: review?(jonas) → review?(bent.mozilla)
Comment on attachment 654475 [details] [diff] [review]
use nullptr

Review of attachment 654475 [details] [diff] [review]:
-----------------------------------------------------------------

We might need to set an exception, but it doesn't look like this code tends to be reached.

In any case, this change is correct.
Attachment #654475 - Flags: review?(bent.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/2995a63cf399
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.