Closed Bug 865503 Opened 7 years ago Closed 7 years ago
Remove unnecessary else-after-return from ns
Table Frame::Ancestors Have Style Height
I noticed an unnecessary else-after-return in nsTableFrame::AncestorsHaveStyleHeight. Patch coming up.
This patch just drops the unnecessary 'else'. [I initially was thinking we could drop this line entirely and just fall down to the existing final "return false", but that doesn't work because this code is inside of a "for" loop, which loops back around before hitting that final return statement.]
Actually, we might as well just directly return the boolean condition, rather than checking the condition and returning true if it's true, false if it's false.
Comment on attachment 741607 [details] [diff] [review] fix v2 r=mats
Attachment #741607 - Flags: review?(matspal) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.