Author: noelgrandin
Date: Mon Oct 24 14:29:48 2011
New Revision: 1188153

URL: http://svn.apache.org/viewvc?rev=1188153&view=rev
Log:
cleanup: a boolean is good enough for a progress flag, don't need an int counter

Modified:
    pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TablePaneSkin.java

Modified: pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TablePaneSkin.java
URL: 
http://svn.apache.org/viewvc/pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TablePaneSkin.java?rev=1188153&r1=1188152&r2=1188153&view=diff
==============================================================================
--- pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TablePaneSkin.java (original)
+++ pivot/trunk/wtk/src/org/apache/pivot/wtk/skin/TablePaneSkin.java Mon Oct 24 
14:29:48 2011
@@ -1512,9 +1512,9 @@ public class TablePaneSkin extends Conta
 
         // If we have don't actually have enough height available
 
-        int progress = 1; // prevent infinite loop
-        while (remainingHeight < 0 && progress > 0) {
-            progress = 0;
+        boolean progress = true; // prevent infinite loop
+        while (remainingHeight < 0 && progress) {
+            progress = false;
             for (int i = 0; i < rowCount; i++) {
                 if (isRowVisible(i)) {
                     TablePane.Row row = rows.get(i);
@@ -1522,7 +1522,7 @@ public class TablePaneSkin extends Conta
                         if (rowHeights[i] > 0) {
                             rowHeights[i]--;
                             remainingHeight++;
-                            progress++;
+                            progress = true;
                             if (remainingHeight >= 0) break;
                         }
                     }


Reply via email to