# HG changeset patch
# User skidd13 <skidd13@openttd.org>
# Date 2007-11-28 21:59:06
# Node ID bd7f4c5a45b3e886ba322e45aa11f7d1b6c608e6
# Parent  2dc6f47ed3a54e001626a3c607acf7d7aba05586

(svn r11538) -Codechange: Rewrite GetNthSetBit in a more uncontroversial way and add its documentation

diff --git a/src/town_gui.cpp b/src/town_gui.cpp
--- a/src/town_gui.cpp
+++ b/src/town_gui.cpp
@@ -109,15 +109,22 @@ uint GetMaskOfTownActions(int *nump, Pla
 	return buttons;
 }
 
+/**
+ * Get the position of the Nth set bit.
+ *
+ * If there is no Nth bit set return -1
+ *
+ * @param bits The value to search in
+ * @param n The Nth set bit from which we want to know the position
+ * @return The position of the Nth set bit
+ */
 static int GetNthSetBit(uint32 bits, int n)
 {
-	int i = 0;
-
 	if (n >= 0) {
-		do {
-			if (bits & 1 && --n < 0) return i;
-			i++;
-		} while (bits >>= 1);
+		for (uint i = 0; bits != 0; bits >>= 1, i++) {
+			if (bits & 1) n--;
+			if (n < 0) return i;
+		}
 	}
 	return -1;
 }