Make sure the for loop terminates. Also caught by bryner, r+sr=me

This commit is contained in:
bzbarsky%mit.edu 2003-07-07 05:15:41 +00:00
Родитель 5c861d776b
Коммит 5e0a1a7710
2 изменённых файлов: 2 добавлений и 2 удалений

Просмотреть файл

@ -285,7 +285,7 @@ nsTableRowFrame::InsertFrames(nsIPresContext* aPresContext,
nsTableCellFrame* prevCellFrame = (nsTableCellFrame *)nsTableFrame::GetFrameAtOrBefore(aPresContext, this, aPrevFrame, cellFrameType); nsTableCellFrame* prevCellFrame = (nsTableCellFrame *)nsTableFrame::GetFrameAtOrBefore(aPresContext, this, aPrevFrame, cellFrameType);
nsVoidArray cellChildren; nsVoidArray cellChildren;
for (nsIFrame* childFrame = aFrameList; childFrame; for (nsIFrame* childFrame = aFrameList; childFrame;
childFrame->GetNextSibling()) { childFrame = childFrame->GetNextSibling()) {
nsCOMPtr<nsIAtom> frameType; nsCOMPtr<nsIAtom> frameType;
childFrame->GetFrameType(getter_AddRefs(frameType)); childFrame->GetFrameType(getter_AddRefs(frameType));
if (IS_TABLE_CELL(frameType.get())) { if (IS_TABLE_CELL(frameType.get())) {

Просмотреть файл

@ -285,7 +285,7 @@ nsTableRowFrame::InsertFrames(nsIPresContext* aPresContext,
nsTableCellFrame* prevCellFrame = (nsTableCellFrame *)nsTableFrame::GetFrameAtOrBefore(aPresContext, this, aPrevFrame, cellFrameType); nsTableCellFrame* prevCellFrame = (nsTableCellFrame *)nsTableFrame::GetFrameAtOrBefore(aPresContext, this, aPrevFrame, cellFrameType);
nsVoidArray cellChildren; nsVoidArray cellChildren;
for (nsIFrame* childFrame = aFrameList; childFrame; for (nsIFrame* childFrame = aFrameList; childFrame;
childFrame->GetNextSibling()) { childFrame = childFrame->GetNextSibling()) {
nsCOMPtr<nsIAtom> frameType; nsCOMPtr<nsIAtom> frameType;
childFrame->GetFrameType(getter_AddRefs(frameType)); childFrame->GetFrameType(getter_AddRefs(frameType));
if (IS_TABLE_CELL(frameType.get())) { if (IS_TABLE_CELL(frameType.get())) {