Closed Bug 687121 Opened 8 years ago Closed 8 years ago

GCC 4.6 build warning: "js/src/xpconnect/shell/xpcshell.cpp:549: warning: variable 'preBytes' set but not used [-Wunused-but-set-variable]"

Categories

(Core :: XPConnect, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: froydnj, Assigned: froydnj)

References

(Blocks 1 open bug)

Details

(Whiteboard: [build_warning])

Attachments

(1 file, 1 obsolete file)

The `preBytes' variable in xpcshell.cpp:GC is set but not used.  Patch coming up.
Attached patch patch to remove preBytes (obsolete) — Splinter Review
Attachment #560591 - Flags: review?(jorendorff)
Assignee: nobody → nfroyd
Blocks: buildwarning
Whiteboard: [build_warning]
Comment on attachment 560591 [details] [diff] [review]
patch to remove preBytes

OK.
Attachment #560591 - Flags: review?(jorendorff) → review+
Comment on attachment 560591 [details] [diff] [review]
patch to remove preBytes

Tweak name of patch.
Attachment #560591 - Attachment description: patch to removed preBytes → patch to remove preBytes
Keywords: checkin-needed
Updated patch with proper commit message.
Attachment #560591 - Attachment is obsolete: true
Attachment #565248 - Flags: review+
In my queue, which is heading to try first then inbound :-)
https://tbpl.mozilla.org/?tree=Try&rev=a8fbb2a76633
Keywords: checkin-needed
OS: Linux → All
Hardware: x86 → All
https://hg.mozilla.org/integration/mozilla-inbound/rev/69ae30e1bc3e
Status: NEW → ASSIGNED
Flags: in-testsuite-
Target Milestone: --- → mozilla10
Version: unspecified → Trunk
https://hg.mozilla.org/mozilla-central/rev/69ae30e1bc3e
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.