cpp/openttd-patchpack/source Changeset - r7316:32b8e520559e

Changeset - r7316:32b8e520559e

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

truelight - 17 years ago 2007-07-24 12:29:47
truelight@openttd.org

(svn r10671) -Codechange: don't mix both lookup and temp-variable-with-value-of-lookup (skidd13)

1 file changed with 4 insertions and 6 deletions:

src/strings.cpp

0 comments (0 inline, 0 general)

src/strings.cpp

➞

Show inline comments

@@ @@ -241,7 +241,7 @@ static const uint32 _divisor_table[] = { @@
 // TODO
 static char *FormatCommaNumber(char *buff, int32 number, const char* last)
+{
-	uint32 quot,divisor;
 	uint32 quot;
 	int i;
 	uint32 tot;
 	uint32 num;
@@ @@ -255,9 +255,8 @@ static char *FormatCommaNumber(char *buf @@
 	tot = 0;
 	for (i = 0; i != 10; i++) {
 		divisor = _divisor_table[i];
 		quot = 0;
-		if (num >= divisor) {
+		if (num >= _divisor_table[i]) {
 			quot = num / _divisor_table[i];
 			num = num % _divisor_table[i];
+		}
@@ @@ -275,7 +274,7 @@ static char *FormatCommaNumber(char *buf @@
 // TODO
 static char *FormatNoCommaNumber(char *buff, int32 number, const char* last)
+{
-	uint32 quot,divisor;
 	uint32 quot;
 	int i;
 	uint32 tot;
 	uint32 num;
@@ @@ -289,9 +288,8 @@ static char *FormatNoCommaNumber(char *b @@
 	tot = 0;
 	for (i = 0; i != 10; i++) {
 		divisor = _divisor_table[i];
 		quot = 0;
-		if (num >= divisor) {
+		if (num >= _divisor_table[i]) {
 			quot = num / _divisor_table[i];
 			num = num % _divisor_table[i];
+		}

0 comments (0 inline, 0 general)