diff -r 6171f0bad318 -r f7b49b2c5d84 project_files/frontlib/ipc/ipcbase.c --- a/project_files/frontlib/ipc/ipcbase.c Wed Jun 27 22:52:19 2012 +0200 +++ b/project_files/frontlib/ipc/ipcbase.c Thu Jul 05 00:33:24 2012 +0200 @@ -64,8 +64,7 @@ } uint16_t flib_ipcbase_port(flib_ipcbase *ipc) { - if(!ipc) { - flib_log_e("null parameter in flib_ipcbase_port"); + if(log_badargs_if(ipc==NULL)) { return 0; } return ipc->port; @@ -80,8 +79,7 @@ } IpcState flib_ipcbase_state(flib_ipcbase *ipc) { - if(!ipc) { - flib_log_e("null parameter in flib_ipcbase_state"); + if(log_badargs_if(ipc==NULL)) { return IPC_NOT_CONNECTED; } else if(ipc->sock) { return IPC_CONNECTED; @@ -109,8 +107,7 @@ } int flib_ipcbase_recv_message(flib_ipcbase *ipc, void *data) { - if(!ipc || !data) { - flib_log_e("null parameter in flib_ipcbase_recv_message"); + if(log_badargs_if2(ipc==NULL, data==NULL)) { return -1; } @@ -134,8 +131,7 @@ } int flib_ipcbase_recv_map(flib_ipcbase *ipc, void *data) { - if(!ipc || !data) { - flib_log_e("null parameter in flib_ipcbase_recv_map"); + if(log_badargs_if2(ipc==NULL, data==NULL)) { return -1; } @@ -167,15 +163,10 @@ } int flib_ipcbase_send_raw(flib_ipcbase *ipc, const void *data, size_t len) { - if(!ipc || (!data && len>0)) { - flib_log_e("null parameter in flib_ipcbase_send_raw"); + if(log_badargs_if2(ipc==NULL, data==NULL && len>0) + || log_w_if(!ipc->sock, "flib_ipcbase_send_raw: Not connected.")) { return -1; } - if(!ipc->sock) { - flib_log_w("flib_ipcbase_send_raw: Not connected."); - return -1; - } - if(flib_socket_send(ipc->sock, data, len) == len) { logSentMsg(data, len); return 0; @@ -188,11 +179,7 @@ } int flib_ipcbase_send_message(flib_ipcbase *ipc, void *data, size_t len) { - if(!ipc || (!data && len>0)) { - flib_log_e("null parameter in flib_ipcbase_send_message"); - return -1; - } else if(len>255) { - flib_log_e("Overlong message (%zu bytes) in flib_ipcbase_send_message", len); + if(log_badargs_if3(ipc==NULL, data==NULL && len>0, len>255)) { return -1; } @@ -203,9 +190,7 @@ } void flib_ipcbase_accept(flib_ipcbase *ipc) { - if(!ipc) { - flib_log_e("null parameter in flib_ipcbase_accept"); - } else if(!ipc->sock && ipc->acceptor) { + if(!log_badargs_if(ipc==NULL) && !ipc->sock && ipc->acceptor) { ipc->sock = flib_socket_accept(ipc->acceptor, true); if(ipc->sock) { flib_acceptor_close(ipc->acceptor);