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

RESOLVED FIXED

Status

()

Core
Layout: Text
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

Trunk
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [build_warning], URL)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 527088 [details] [diff] [review]
fix

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+
(Assignee)

Comment 1

6 years ago
pushed: http://hg.mozilla.org/mozilla-central/rev/7ad20fbb9992
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.