[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdelibs/khtml/rendering
From:       David Faure <faure () kde ! org>
Date:       2005-04-18 11:09:55
Message-ID: 20050418110955.79384625 () office ! kde ! org
[Download RAW message or body]

CVS commit by faure: 

Fix compiler warnings about unused params - increases code readability.


  M +12 -12    render_box.cpp   1.260
  M +10 -10    render_table.cpp   1.279
  M +3 -3      table_layout.cpp   1.42
  M +1 -1      table_layout.h   1.8


--- kdelibs/khtml/rendering/render_box.cpp  #1.259:1.260
@@ -1449,15 +1449,15 @@ void RenderBox::calcAbsoluteVertical()
 
 
-int RenderBox::lowestPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::lowestPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     return includeSelf ? m_height : 0;
 }
 
-int RenderBox::rightmostPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::rightmostPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     return includeSelf ? m_width : 0;
 }
 
-int RenderBox::leftmostPosition(bool includeOverflowInterior, bool includeSelf) const
+int RenderBox::leftmostPosition(bool /*includeOverflowInterior*/, bool includeSelf) const
 {
     return includeSelf ? 0 : m_width;

--- kdelibs/khtml/rendering/render_table.cpp  #1.278:1.279
@@ -2392,5 +2392,5 @@ class CollapsedBorders
 {
 public:
-    CollapsedBorders(int i) :count(0) {}
+    CollapsedBorders() :count(0) {}
 
     void addBorder(const CollapsedBorderValue& b, RenderObject::BorderSide s, bool paint,
@@ -2490,5 +2490,5 @@ void RenderTableCell::paintCollapsedBord
     // We never paint diagonals at the joins.  We simply let the border with the highest
     // precedence paint on top of borders with lower precedence.
-    CollapsedBorders borders(4);
+    CollapsedBorders borders;
     borders.addBorder(topVal, BSTop, render_t, _tx, _ty, _tx + w, _ty + topWidth, ts);
     borders.addBorder(bottomVal, BSBottom, render_b, _tx, _ty + h - bottomWidth, _tx + w, _ty + h, bs);

--- kdelibs/khtml/rendering/table_layout.cpp  #1.41:1.42
@@ -88,5 +88,5 @@ FixedTableLayout::~FixedTableLayout()
 }
 
-int FixedTableLayout::calcWidthArray( int tableWidth )
+int FixedTableLayout::calcWidthArray()
 {
     int usedWidth = 0;
@@ -100,5 +100,5 @@ int FixedTableLayout::calcWidthArray( in
 
 #ifdef DEBUG_LAYOUT
-    qDebug("FixedTableLayout::calcWidthArray( %d )", tableWidth );
+    qDebug("FixedTableLayout::calcWidthArray()" );
     qDebug("    col elements:");
 #endif
@@ -236,5 +236,5 @@ void FixedTableLayout::calcMinMaxWidth()
     int tableWidth = table->style()->width().isFixed() ? table->style()->width().value() - bs : 0;
 
-    int mw = calcWidthArray( tableWidth ) + bs;
+    int mw = calcWidthArray() + bs;
     table->m_minWidth = kMin( kMax( mw, tableWidth ), 0x7fff );
     table->m_maxWidth = table->m_minWidth;

--- kdelibs/khtml/rendering/table_layout.h  #1.7:1.8
@@ -59,5 +59,5 @@ public:
 
 protected:
-    int calcWidthArray( int tableWidth );
+    int calcWidthArray();
 
     QMemArray<Length> width;


[prev in list] [next in list] [prev in thread] [next in thread]