Mercurial > hg > openttd

changeset 14523:f475a3b90d22 draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
(svn r19094) -Codechange: don't check for NULL values before calling StrEmpty
author yexo <yexo@openttd.org>
date Thu, 11 Feb 2010 14:58:51 +0000
parents 6db23c86f0a9
children 02ddea71c644
files src/blitter/factory.hpp src/network/network_udp.cpp src/newgrf_gui.cpp
diffstat 3 files changed, 6 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/src/blitter/factory.hpp
+++ b/src/blitter/factory.hpp
@@ -106,7 +106,7 @@
 				delete *GetActiveBlitter();
 				*GetActiveBlitter() = newb;
 
-				DEBUG(driver, 1, "Successfully %s blitter '%s'",StrEmpty(name) ? "probed" : "loaded", bname);
+				DEBUG(driver, 1, "Successfully %s blitter '%s'", StrEmpty(name) ? "probed" : "loaded", bname);
 				return newb;
 			}
 		}
--- a/src/network/network_udp.cpp
+++ b/src/network/network_udp.cpp
@@ -190,7 +190,7 @@
 		 * the current list and do not send the other data.
 		 * The name could be an empty string, if so take the filename. */
 		packet_len += sizeof(c.grfid) + sizeof(c.md5sum) +
-				min(strlen((f->name != NULL && !StrEmpty(f->name)) ? f->name : f->filename) + 1, (size_t)NETWORK_GRF_NAME_LENGTH);
+				min(strlen((!StrEmpty(f->name)) ? f->name : f->filename) + 1, (size_t)NETWORK_GRF_NAME_LENGTH);
 		if (packet_len > SEND_MTU - 4) { // 4 is 3 byte header + grf count in reply
 			break;
 		}
@@ -206,8 +206,7 @@
 		char name[NETWORK_GRF_NAME_LENGTH];
 
 		/* The name could be an empty string, if so take the filename */
-		strecpy(name, (in_reply[i]->name != NULL && !StrEmpty(in_reply[i]->name)) ?
-				in_reply[i]->name : in_reply[i]->filename, lastof(name));
+		strecpy(name, (!StrEmpty(in_reply[i]->name)) ? in_reply[i]->name : in_reply[i]->filename, lastof(name));
 		this->Send_GRFIdentifier(&packet, in_reply[i]);
 		packet.Send_string(name);
 	}
--- a/src/newgrf_gui.cpp
+++ b/src/newgrf_gui.cpp
@@ -131,7 +131,7 @@
 	if (HasBit(c->flags, GCF_COMPATIBLE)) y = DrawStringMultiLine(x, right, y, bottom, STR_NEWGRF_COMPATIBLE_LOADED);
 
 	/* Draw GRF info if it exists */
-	if (c->info != NULL && !StrEmpty(c->info)) {
+	if (!StrEmpty(c->info)) {
 		SetDParam(0, STR_JUST_RAW_STRING);
 		SetDParamStr(1, c->info);
 		y = DrawStringMultiLine(x, right, y, bottom, STR_BLACK_STRING);
@@ -326,7 +326,7 @@
 				{
 					const GRFConfig *c = this->grfs[i];
 					bool h = c == this->sel;
-					const char *text = (c->name != NULL && !StrEmpty(c->name)) ? c->name : c->filename;
+					const char *text = (!StrEmpty(c->name)) ? c->name : c->filename;
 
 					/* Draw selection background */
 					if (h) GfxFillRect(r.left + 1, y, r.right - 1, y + this->resize.step_height - 1, 156);
@@ -657,7 +657,7 @@
 				int i = 0;
 				for (const GRFConfig *c = this->list; c != NULL; c = c->next, i++) {
 					if (this->vscroll.IsVisible(i)) {
-						const char *text = (c->name != NULL && !StrEmpty(c->name)) ? c->name : c->filename;
+						const char *text = (!StrEmpty(c->name)) ? c->name : c->filename;
 						PaletteID pal;
 
 						/* Pick a colour */