diff -r 6171f0bad318 -r f7b49b2c5d84 project_files/frontlib/net/netbase.c --- a/project_files/frontlib/net/netbase.c Wed Jun 27 22:52:19 2012 +0200 +++ b/project_files/frontlib/net/netbase.c Thu Jul 05 00:33:24 2012 +0200 @@ -36,6 +36,10 @@ }; flib_netbase *flib_netbase_create(const char *server, uint16_t port) { + if(log_badargs_if2(server==NULL, port==0)) { + return NULL; + } + flib_netbase *result = NULL; flib_netbase *newNet = flib_calloc(1, sizeof(flib_netbase)); @@ -62,14 +66,10 @@ } bool flib_netbase_connected(flib_netbase *net) { - if(!net) { - flib_log_e("null parameter in flib_netbase_connected"); - return false; - } else if(net->sock) { + if(!log_badargs_if(net==NULL) && net->sock) { return true; - } else { - return false; } + return false; } /** @@ -135,8 +135,7 @@ } flib_netmsg *flib_netbase_recv_message(flib_netbase *net) { - if(!net) { - flib_log_e("null parameter in flib_netbase_recv_message"); + if(log_badargs_if(net==NULL)) { return NULL; } @@ -160,8 +159,7 @@ } int flib_netbase_send_raw(flib_netbase *net, const void *data, size_t len) { - if(!net || (!data && len>0)) { - flib_log_e("null parameter in flib_netbase_send_raw"); + if(log_badargs_if2(net==NULL, data==NULL && len>0)) { return -1; } if(!net->sock) { @@ -181,8 +179,7 @@ } int flib_netbase_send_message(flib_netbase *net, flib_netmsg *msg) { - if(!net || !msg) { - flib_log_e("null parameter in flib_netbase_send_message"); + if(log_badargs_if2(net==NULL, msg==NULL)) { return -1; } @@ -209,9 +206,7 @@ int flib_netbase_sendf(flib_netbase *net, const char *format, ...) { int result = -1; - if(!net || !format) { - flib_log_e("null parameter in flib_netbase_sendf"); - } else { + if(!log_badargs_if2(net==NULL, format==NULL)) { va_list argp; va_start(argp, format); char *buffer = flib_vasprintf(format, argp); @@ -247,9 +242,7 @@ int flib_netmsg_append_part(flib_netmsg *msg, const void *part, size_t partlen) { int result = -1; - if(!msg) { - flib_log_e("null parameter in flib_netmsg_append_part"); - } else { + if(!log_badargs_if2(msg==NULL, part==NULL && partlen>0)) { char **newParts = realloc(msg->parts, (msg->partCount+1)*sizeof(*msg->parts)); if(newParts) { msg->parts = newParts;