Closed Bug 1008908 Opened 10 years ago Closed 10 years ago

Useless assignment to 'remainingISize'

Categories

(Core :: Layout: Block and Inline, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

Details

Attachments

(1 file)

The value of this assignment to 'remainingISize' is never used:
http://hg.mozilla.org/mozilla-central/annotate/3bb70c0d6845/layout/generic/nsLineLayout.cpp#l2579

There used to be a use of 'remainingISize' after the switch but it
was removed here:
http://hg.mozilla.org/mozilla-central/diff/aeff4052ef00/layout/generic/nsLineLayout.cpp#l1.2526
(it was called 'remainingWidth' back then)
Attached patch fixSplinter Review
(removed a superfluous blank line while I'm here)
Attachment #8421236 - Flags: review?(smontagu)
Attachment #8421236 - Flags: review?(smontagu) → review+
Keywords: checkin-needed
Hi,
could you provide a Try link. Suggestions for what to run if you haven't
yet can be found here:
https://wiki.mozilla.org/Sheriffing/How:To:Recommended_Try_Practices
https://hg.mozilla.org/mozilla-central/rev/5537f0724a22
Assignee: nobody → matspal
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: