diff options
author | tron <tron@openttd.org> | 2007-01-13 15:00:16 +0000 |
---|---|---|
committer | tron <tron@openttd.org> | 2007-01-13 15:00:16 +0000 |
commit | e043c6cef7b8afb694a528218c18a15bae3b5315 (patch) | |
tree | 9bb1adc7db26f0d3e419320319b33f692cbcb593 /src/network/network_udp.cpp | |
parent | 64f34d322d806e4ff3404b7d4f59a63aed5a82d3 (diff) | |
download | openttd-e043c6cef7b8afb694a528218c18a15bae3b5315.tar.xz |
(svn r8097) Replace strlen() {==,!=,>} 0 by the more concise {,!}StrEmpty(). Additionally the test takes O(1) instead of O(n) now
Diffstat (limited to 'src/network/network_udp.cpp')
-rw-r--r-- | src/network/network_udp.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/network/network_udp.cpp b/src/network/network_udp.cpp index f6ef19fe3..08a145717 100644 --- a/src/network/network_udp.cpp +++ b/src/network/network_udp.cpp @@ -238,7 +238,7 @@ DEF_UDP_RECEIVE_COMMAND(Server, PACKET_UDP_CLIENT_GET_NEWGRFS) * 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 && strlen(f->name) > 0) ? f->name : f->filename) + 1, (size_t)NETWORK_GRF_NAME_LENGTH); + min(strlen((f->name != NULL && !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; } @@ -254,7 +254,7 @@ DEF_UDP_RECEIVE_COMMAND(Server, PACKET_UDP_CLIENT_GET_NEWGRFS) char name[NETWORK_GRF_NAME_LENGTH]; /* The name could be an empty string, if so take the filename */ - ttd_strlcpy(name, (in_reply[i]->name != NULL && strlen(in_reply[i]->name) > 0) ? + ttd_strlcpy(name, (in_reply[i]->name != NULL && !StrEmpty(in_reply[i]->name)) ? in_reply[i]->name : in_reply[i]->filename, sizeof(name)); this->Send_GRFIdentifier(packet, in_reply[i]); NetworkSend_string(packet, name); @@ -395,7 +395,7 @@ DEF_UDP_RECEIVE_COMMAND(Client, PACKET_UDP_SERVER_NEWGRFS) /* An empty name is not possible under normal circumstances * and causes problems when showing the NewGRF list. */ - if (strlen(name) == 0) continue; + if (StrEmpty(name)) continue; /* Finds the fake GRFConfig for the just read GRF ID and MD5sum tuple. * If it exists and not resolved yet, then name of the fake GRF is |