Closed Bug 650987 Opened 10 years ago Closed 10 years ago
Cache Update Parent .cpp:43: warning: 'g Offline Cache Update Service' defined but not used
Filing bug on these build warnings: > uriloader/prefetch/OfflineCacheUpdateParent.cpp:43: warning: 'gOfflineCacheUpdateService' defined but not used > uriloader/prefetch/OfflineCacheUpdateChild.cpp:65: warning: 'gOfflineCacheUpdateService' defined but not used I think this is a copy-paste error, from grabbing code from nsOfflineCacheUpdate.cpp including this unused variable, and then the copies-of-the-unused-variable remained even after nsOfflineCacheUpdate.cpp was fixed (in bug 615425). Attached patch fixes this.
Attachment #526875 - Flags: review?(honzab.moz)
--> adding clobber build log including these warnings to URL field.
The patch also adds the formerly-missing newline at the end of these files. (Bugzilla's patch viewer doesn't show that correctly, but the raw text of the patch does.)
10 years ago
Attachment #526875 - Flags: review?(honzab.moz) → review+
Here's the changeset for checkin. (I can't treewatch at the moment; hoping some kind soul can land this & bug 650871 as ridealongs)
Status: NEW → ASSIGNED
Whiteboard: [build_warning] → [build_warning][fixed in cedar]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [build_warning][fixed in cedar] → [build_warning]
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.