Closed Bug 654371 Opened 13 years ago Closed 13 years ago

nsOggReader.cpp:1065:18: warning: unused variable ‘range’

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: dholbert, Unassigned)

References

Details

(Whiteboard: [build_warning])

Filing bug on this build warning:

> nsOggReader.cpp: In member function ‘nsresult nsOggReader::GetSeekRanges(nsTArray<nsOggReader::SeekRange>&)’:
> nsOggReader.cpp:1065:18: warning: unused variable ‘range’

This variable is just an alias for cached[index], but it looks like we never used the alias even in the cset that added it:
  http://hg.mozilla.org/mozilla-central/rev/d3aeb102da2b

cpearce, perhaps you intended to replace the "cached[index]" usages with the alias?  Or, should this variable just go away?
Here's the link to the flagged line in MXR, for reference:
http://mxr.mozilla.org/mozilla-central/source/content/media/ogg/nsOggReader.cpp#1065
Thanks, I've got a fix for this included in my patch in bug 650994.
Depends on: 650994
Fixed by landing of bug 650994.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.