The default bug view has changed. See this FAQ.

Unnecessary nsresult rv declaration in nsFileChannel::OpenContentStream

RESOLVED FIXED in mozilla10

Status

()

Core
Networking: File
--
trivial
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: WeirdAl, Assigned: WeirdAl)

Tracking

Trunk
mozilla10
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 570526 [details] [diff] [review]
patch

Near the end of nsFileChannel::OpenContentStream, there's an inner nsresult rv declaration.  We already have an available nsresult rv though, and it's not checked anywhere else.
Attachment #570526 - Flags: review?(bzbarsky)
Comment on attachment 570526 [details] [diff] [review]
patch

r=me
Attachment #570526 - Flags: review?(bzbarsky) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → ajvincent
http://hg.mozilla.org/integration/mozilla-inbound/rev/88cd8e9287c8
Keywords: checkin-needed
Target Milestone: --- → mozilla10
https://hg.mozilla.org/mozilla-central/rev/88cd8e9287c8
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.