Closed Bug 611396 Opened 15 years ago Closed 15 years ago

StartupCache.cpp has 3 "warning: unused variable 'rv'"

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: dholbert, Unassigned)

Details

(Whiteboard: [build_warning])

Attachments

(1 file)

Attached patch trivial fixSplinter Review
StartupCache.cpp currently has this build-warning-spam: { ../../mozilla/startupcache/StartupCache.cpp: In member function 'virtual nsresult mozilla::scache::StartupCacheDebugOutputStream::WriteObject(nsISupports*, PRBool)': ../../mozilla/startupcache/StartupCache.cpp:487: warning: unused variable 'rv' ../../mozilla/startupcache/StartupCache.cpp: In member function 'virtual nsresult mozilla::scache::StartupCacheDebugOutputStream::WriteSingleRefObject(nsISupports*)': ../../mozilla/startupcache/StartupCache.cpp:501: warning: unused variable 'rv' ../../mozilla/startupcache/StartupCache.cpp: In member function 'virtual nsresult mozilla::scache::StartupCacheDebugOutputStream::WriteCompoundObject(nsISupports*, const nsIID&, PRBool)': ../../mozilla/startupcache/StartupCache.cpp:516: warning: unused variable 'rv' } It's just from three completely-unused "nsresult rv;" lines. Attached patch fixes this.
Attachment #489861 - Flags: review?
Attachment #489861 - Flags: review? → review?(dwitte)
Comment on attachment 489861 [details] [diff] [review] trivial fix If it builds, ship it.
Attachment #489861 - Flags: review?(dwitte) → review+
Comment on attachment 489861 [details] [diff] [review] trivial fix Yay, thanks! (yes, it does build) Need approval to land, though -- requesting approval. Justification: trivial patch, no risk. As noted in comment 0, this just removes three absolutely-unnecessary lines of the form: > nsresult rv;
Attachment #489861 - Flags: approval2.0?
Attachment #489861 - Flags: approval2.0? → approval2.0+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: