Mercurial > hg > openttd
changeset 6062:2708f929bb6b draft
(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;