The default bug view has changed. See this FAQ.

nsObjectLoadingContent.cpp:2501:11: warning: unused variable ‘topUri’ [-Wunused-variable]

RESOLVED FIXED in mozilla17

Status

()

Core
Plug-ins
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dholbert, Assigned: johns)

Tracking

(Blocks: 1 bug)

Trunk
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Newish build warning:
{
content/base/src/nsObjectLoadingContent.cpp: In member function ‘bool nsObjectLoadingContent::ShouldPlay(nsObjectLoadingContent::FallbackType&)’:

content/base/src/nsObjectLoadingContent.cpp:2501:11: warning: unused variable ‘topUri’ [-Wunused-variable]
}

This variable was never unused in the changeset that added it (for bug 745030):
  https://hg.mozilla.org/mozilla-central/rev/615a657305b5#l1.1453

Presumably, that line can just be removed.
(Reporter)

Comment 1

5 years ago
(In reply to Daniel Holbert [:dholbert] from comment #0)
> This variable was never unused in the changeset that added it

er "never used"
(Assignee)

Updated

5 years ago
Assignee: nobody → jschoenick
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: x86_64 → All
(Assignee)

Comment 2

5 years ago
Created attachment 652592 [details] [diff] [review]
Remove unused variable
Attachment #652592 - Flags: review?(joshmoz)

Updated

5 years ago
Attachment #652592 - Flags: review?(joshmoz) → review+
(Assignee)

Comment 3

5 years ago
Comment on attachment 652592 [details] [diff] [review]
Remove unused variable

https://hg.mozilla.org/integration/mozilla-inbound/rev/57de8dafdf2d

try:
https://tbpl.mozilla.org/?tree=Try&rev=46b32a4de584
Attachment #652592 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/57de8dafdf2d
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.