From 3c5eb8458a0f56fdc7eb7574e96e30e4431a2c67 Mon Sep 17 00:00:00 2001
From: maedhros <maedhros@openttd.org>
Date: Sat, 17 Feb 2007 23:01:42 +0000
Subject: [PATCH] (svn r8792) -Fix (r6623): Don't check whether a string length
 is 1 when you already know it isn't.

---
 src/newgrf.cpp | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/newgrf.cpp b/src/newgrf.cpp
index e1ba368b77..afaa760bc7 100644
--- a/src/newgrf.cpp
+++ b/src/newgrf.cpp
@@ -2074,7 +2074,11 @@ static void FeatureNewName(byte *buf, int len)
 	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 @@ static void FeatureNewName(byte *buf, int len)
 					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;