Mercurial > hg > openttd

changeset 6062:2708f929bb6b draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
(svn r8792) -Fix (r6623): Don't check whether a string length is 1 when you already know it isn't.
author maedhros <maedhros@openttd.org>
date Sat, 17 Feb 2007 23:01:42 +0000
parents c1fc6b10db93
children 555c37d7c315
files src/newgrf.cpp
diffstat 1 files changed, 5 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/src/newgrf.cpp
+++ b/src/newgrf.cpp
@@ -2074,7 +2074,11 @@
 	for (; id < endid && len > 0; id++) {
 		size_t ofs = strlen(name) + 1;
 
-		if (ofs < 128) {
+		if (ofs == 1) {
+			grfmsg(7, "FeatureNewName: Can't add empty name");
+		} else if (ofs > 127) {
+			grfmsg(7, "FeatureNewName: Too long a name (%d)", ofs);
+		} else {
 			grfmsg(8, "FeatureNewName: %d <- %s", id, name);
 
 			switch (feature) {
@@ -2143,14 +2147,6 @@
 					break;
 #endif
 			}
-		} else {
-			/* ofs is the string length + 1, so if the string is empty, ofs
-			 * is 1 */
-			if (ofs == 1) {
-				grfmsg(7, "FeatureNewName: Can't add empty name");
-			} else {
-				grfmsg(7, "FeatureNewName: Too long a name (%d)", ofs);
-			}
 		}
 		name += ofs;
 		len -= (int)ofs;