From d4ea1ed1b64f379bcdb011f500fac5e086f9b5d0 Mon Sep 17 00:00:00 2001
From: "buster%netscape.com"
 <buster%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Tue, 20 Oct 1998 14:35:41 +0000
Subject: [PATCH] minor fix, moved a method call outside an assert macro

git-svn-id: svn://10.0.0.236/trunk@13198 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/layout/html/table/src/nsCellMap.cpp | 6 +++---
 mozilla/layout/tables/nsCellMap.cpp         | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/mozilla/layout/html/table/src/nsCellMap.cpp b/mozilla/layout/html/table/src/nsCellMap.cpp
index d0238a36502..ef50a6aa137 100644
--- a/mozilla/layout/html/table/src/nsCellMap.cpp
+++ b/mozilla/layout/html/table/src/nsCellMap.cpp
@@ -336,7 +336,8 @@ PRBool nsCellMap::RowHasSpanningCells(PRInt32 aRowIndex)
   */
 PRBool nsCellMap::ColIsSpannedInto(PRInt32 aColIndex)
 {
-  NS_PRECONDITION (0<=aColIndex && aColIndex<GetColCount(), "bad col index arg");
+  PRInt32 colCount = GetColCount();
+  NS_PRECONDITION (0<=aColIndex && aColIndex<colCount, "bad col index arg");
   PRBool result = PR_FALSE;
   PRInt32 rowCount = GetRowCount();
   for (PRInt32 rowIndex=0; rowIndex<rowCount; rowIndex++)
@@ -373,10 +374,9 @@ PRBool nsCellMap::ColIsSpannedInto(PRInt32 aColIndex)
   */
 PRBool nsCellMap::ColHasSpanningCells(PRInt32 aColIndex)
 {
-  NS_PRECONDITION (0<=aColIndex && aColIndex<GetColCount(), "bad col index arg");
   PRBool result = PR_FALSE;
   const PRInt32 colCount = GetColCount();
-  if (aColIndex!=colCount-1)
+  if (aColIndex<colCount-1)
   { // aColIndex is not the last col, so we check the next col after aColIndex for spanners
     const PRInt32 rowCount = GetRowCount();
     for (PRInt32 rowIndex=0; rowIndex<rowCount; rowIndex++)
diff --git a/mozilla/layout/tables/nsCellMap.cpp b/mozilla/layout/tables/nsCellMap.cpp
index d0238a36502..ef50a6aa137 100644
--- a/mozilla/layout/tables/nsCellMap.cpp
+++ b/mozilla/layout/tables/nsCellMap.cpp
@@ -336,7 +336,8 @@ PRBool nsCellMap::RowHasSpanningCells(PRInt32 aRowIndex)
   */
 PRBool nsCellMap::ColIsSpannedInto(PRInt32 aColIndex)
 {
-  NS_PRECONDITION (0<=aColIndex && aColIndex<GetColCount(), "bad col index arg");
+  PRInt32 colCount = GetColCount();
+  NS_PRECONDITION (0<=aColIndex && aColIndex<colCount, "bad col index arg");
   PRBool result = PR_FALSE;
   PRInt32 rowCount = GetRowCount();
   for (PRInt32 rowIndex=0; rowIndex<rowCount; rowIndex++)
@@ -373,10 +374,9 @@ PRBool nsCellMap::ColIsSpannedInto(PRInt32 aColIndex)
   */
 PRBool nsCellMap::ColHasSpanningCells(PRInt32 aColIndex)
 {
-  NS_PRECONDITION (0<=aColIndex && aColIndex<GetColCount(), "bad col index arg");
   PRBool result = PR_FALSE;
   const PRInt32 colCount = GetColCount();
-  if (aColIndex!=colCount-1)
+  if (aColIndex<colCount-1)
   { // aColIndex is not the last col, so we check the next col after aColIndex for spanners
     const PRInt32 rowCount = GetRowCount();
     for (PRInt32 rowIndex=0; rowIndex<rowCount; rowIndex++)