155652 - easy nsTableFrame.cpp build warning fix

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Bernd]

we can easily get of the following warnings:
21.layout/html/table/src/nsTableFrame.cpp:1037 Unused variable `PRInt32 colIndex'
22.layout/html/table/src/nsTableFrame.cpp:1744 Unused variable `nscoord tableY'
23.layout/html/table/src/nsTableFrame.cpp:3235 Unused variable `struct nsMargin
collapseBorder'
layout/html/table/src/nsTableFrame.cpp:3236 Unused variable `struct nsMargin
padding'
layout/html/table/src/nsTableFrame.cpp:3574 Unused variable `nscoord
sumOfRowHeights'
layout/html/table/src/nsTableFrame.cpp:3576 Unused variable `nscoord rowGroupYPos'
27.layout/html/table/src/nsTableFrame.cpp:4872 Unused variable `PRInt32 rgRowIndex'
30.layout/html/table/src/nsTableFrame.cpp:5794 Unused variable `const struct
nsStyleBorder * tableStyle'
31.layout/html/table/src/nsTableFrame.cpp:5814 Unused variable `class
nsTableRowFrame * prevRow'
32.layout/html/table/src/nsTableFrame.cpp:5904 `class nsTableRowFrame *
rowFrame' might be used uninitialized in this function
33.layout/html/table/src/nsTableFrame.cpp:5937 `class nsTableRowFrame *
rowFrame' might be used uninitialized in this function

7.layout/html/table/src/nsTableFrame.cpp:2645 Unused variable `class
nsTableRowGroupFrame * prevRowGroup'

1.layout/html/table/src/nsTableFrame.cpp:7376 `void
GetPlaceholderFor(nsIPresContext &, nsIFrame &, nsIFrame **)' defined but not used

Comment 1

[Bernd]

Attachment: patch

Comment 2

[karnaze (gone)]

Comment on attachment 90110 [details] [diff] [review]
patch

r=karnaze, please remove this line 
+   //nsTableRowGroupFrame* prevRowGroup = (nsTableRowGroupFrame
*)nsTableFrame::GetFrameAtOrBefore(aPresContext, this, aPrevFrame,
nsLayoutAtoms::tableRowGroupFrame);

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 90110 [details] [diff] [review]
patch

Do we ever use GetPlaceholderFor?  Why can't we just remove it?

Also, it's already declared static... do doing it again should not matter...

Comment 4

[Bernd]

Attachment: patch v2

Comment 5

[Bernd]

Comment on attachment 94501 [details] [diff] [review]
patch v2

r=karnaze
carry over from previous patch,

boris you are right that function isnt used anymore

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 94501 [details] [diff] [review]
patch v2

sr=bzbarsky

Comment 7

[Bernd]

fix checked in