DeforaOS/libApp
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Renamed some functions and callbacks for consistency

Browse Source
This commit is contained in:
Pierre Pronchery 2014-03-23 14:36:54 +09:00
parent 515825fa53
commit 2ee7645798
4 changed files with 31 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/App/apptransport.h
View File

@ -73,8 +73,13 @@ struct _AppTransportPluginDefinition
AppTransportPlugin * (*init)(AppTransportPluginHelper * helper,
AppTransportMode mode, char const * name);
void (*destroy)(AppTransportPlugin * transport);
int (*send)(AppTransportPlugin * transport, AppMessage * message);
/* ATM_CLIENT */
int (*client_send)(AppTransportPlugin * transport,
AppMessage * message);
/* ATM_SERVER */
int (*server_send)(AppTransportPlugin * transport,
AppTransportClient * client, AppMessage * message);
};

36
src/apptransport.c
View File

@ -133,22 +133,7 @@ void apptransport_delete(AppTransport * transport)
/* useful */
/* apptransport_client_send */
int apptransport_client_send(AppTransport * transport,
AppTransportClient * client, AppMessage * message)
{
if(transport->mode != ATM_SERVER)
return -error_set_code(1, "%s",
"Only servers can reply to clients");
if(transport->definition->client_send == NULL)
return -error_set_code(1, "%s",
"This transport does not support replies");
return transport->definition->client_send(transport->tplugin, client,
message);
}
/* apptransport_send */
int apptransport_send(AppTransport * transport, AppMessage * message,
int apptransport_client_send(AppTransport * transport, AppMessage * message,
int acknowledge)
{
if(transport->mode == ATM_CLIENT
@ -156,7 +141,22 @@ int apptransport_send(AppTransport * transport, AppMessage * message,
&& acknowledge != 0)
/* FIXME will wrap around after 2^32-1 acknowledgements */
appmessage_set_id(message, ++transport->id);
return transport->definition->send(transport->tplugin, message);
return transport->definition->client_send(transport->tplugin, message);
}
/* apptransport_server_send */
int apptransport_server_send(AppTransport * transport,
AppTransportClient * client, AppMessage * message)
{
if(transport->mode != ATM_SERVER)
return -error_set_code(1, "%s",
"Only servers can reply to clients");
if(transport->definition->server_send == NULL)
return -error_set_code(1, "%s",
"This transport does not support replies");
return transport->definition->server_send(transport->tplugin, client,
message);
}
@ -226,7 +226,7 @@ static int _apptransport_helper_client_receive(AppTransport * transport,
/* XXX we can ignore errors */
if((message = appmessage_new_acknowledgement(id)) != NULL)
{
apptransport_client_send(transport, client, message);
apptransport_server_send(transport, client, message);
appmessage_delete(message);
}
return 0;

10
src/apptransport.h
View File

@ -39,10 +39,12 @@ AppTransport * apptransport_new(AppTransportMode mode,
void apptransport_delete(AppTransport * transport);
/* useful */
int apptransport_client_send(AppTransport * transport,
AppTransportClient * client, AppMessage * message);
int apptransport_send(AppTransport * transport, AppMessage * message,
/* ATM_CLIENT */
int apptransport_client_send(AppTransport * transport, AppMessage * message,
int acknowledge);
/* ATM_SERVER */
int apptransport_server_send(AppTransport * transport,
AppTransportClient * client, AppMessage * message);
#endif /* !LIBAPP_APPTRANSPORT_H */

2
tests/transport.c
View File

@ -207,7 +207,7 @@ static int _transport_callback_idle(void * data)
#ifdef DEBUG
fprintf(stderr, "DEBUG: %s()\n", __func__);
#endif
transport->plugind->send(transport->client, transport->message);
transport->plugind->client_send(transport->client, transport->message);
return 1;
}