Closed Bug 651229 Opened 10 years ago Closed 10 years ago

nsLineBox.cpp:663: warning: unused variable 'presContext'

Categories

(Core :: Layout: Text and Fonts, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dholbert, Assigned: dholbert)

References

()

Details

(Whiteboard: [build_warning])

Attachments

(1 file)

Attached patch fixSplinter Review
Filing bug on this newish build warning:
> layout/generic/nsLineBox.cpp: In member function 'virtual nsresult nsLineIterator::CheckLineOrder(PRInt32, PRBool*, nsIFrame**, nsIFrame**)':
> layout/generic/nsLineBox.cpp:663: warning: unused variable 'presContext'

Looks like the last use of this variable was removed in bug 624798:
>    nsPresContext* presContext = line->mFirstChild->PresContext();
>  
> -  nsBidiPresUtils* bidiUtils = presContext->GetBidiUtils();
> -

Fix attached.
Attachment #527088 - Flags: review?(smontagu)
Attachment #527088 - Flags: review?(smontagu) → review+
pushed: http://hg.mozilla.org/mozilla-central/rev/7ad20fbb9992
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.