The default bug view has changed. See this FAQ.

GCC 4.6 build warning: "nsSocketTransportService2.cpp:681:12: warning: variable ‘pollError’ set but not used [-Wunused-but-set-variable]"

RESOLVED FIXED in mozilla9

Status

()

Core
Networking
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

(Blocks: 1 bug)

Trunk
mozilla9
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
> nsSocketTransportService2.cpp: In member function ‘nsresult nsSocketTransportService::DoPollIteration(PRBool)’:
> nsSocketTransportService2.cpp:681:12: warning: variable ‘pollError’ set but not used [-Wunused-but-set-variable]

This variable has never been used. It was added in this CVS checkin...
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/netwerk/base/src&command=DIFF_FRAMESET&file=nsSocketTransportService2.cpp&rev2=1.21&rev1=1.20
...for bug 326273, back in 2006.

I boldly propose that we remove this variable.
(Assignee)

Comment 1

6 years ago
Created attachment 552719 [details] [diff] [review]
fix: remove variable
Attachment #552719 - Flags: review?(jduell.mcbugs)
Attachment #552719 - Flags: review?(jduell.mcbugs) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/50edc05d6ee8
Assignee: nobody → dholbert
Blocks: 187528
Status: NEW → ASSIGNED
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/50edc05d6ee8
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.