Closed Bug 651567 Opened 10 years ago Closed 10 years ago
Math MLmunder Frame .cpp:285:11: warning: unused variable 'x Height'
Filing bug on this warning: > layout/mathml/nsMathMLmunderFrame.cpp:285:11: warning: unused variable 'xHeight Looks like this variable has actually been unused since it was first added, back in the year 2000: http://mxr.mozilla.org/firefox/source/layout/mathml/base/src/nsMathMLmunderFrame.cpp http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/layout/mathml/base/src&command=DIFF_FRAMESET&file=nsMathMLmunderFrame.cpp&rev2=1.10&rev1=1.9 GCC only recently started warning us about this, though, because this variable had been set by passing it as an outparam -- which looked like we were 'using' it -- up until zwol tweaked it to set it directly, in Bug 174055 part 6. Attached patch kills the variable.
Attachment #527328 - Flags: review?(dbaron)
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #527328 - Flags: review?(dbaron) → review+
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.