diff options
author | Simon Busch <morphis@gravedo.de> | 2012-09-05 09:53:36 +0200 |
---|---|---|
committer | Simon Busch <morphis@gravedo.de> | 2012-09-09 12:23:21 +0200 |
commit | 3a9c878ca37442aca10917306fffb625c46b0ec4 (patch) | |
tree | 0d60e6b6ab27f6e8e7746b3b1abb6f7097f1dc59 /samsung-ipc/ipc.c | |
parent | db180081a182a2cd5652f9b1c147b974bae54811 (diff) | |
download | hardware_replicant_libsamsung-ipc-3a9c878ca37442aca10917306fffb625c46b0ec4.tar.gz hardware_replicant_libsamsung-ipc-3a9c878ca37442aca10917306fffb625c46b0ec4.tar.bz2 hardware_replicant_libsamsung-ipc-3a9c878ca37442aca10917306fffb625c46b0ec4.zip |
ipc: refactor ipc_client_gprs_get_iface method to return iface name instead of using a parameter
Allocating memory and setting a pointer to it for the user via a parameter is kind of
unnecessary. It's better to return the pointer instead.
Diffstat (limited to 'samsung-ipc/ipc.c')
-rw-r--r-- | samsung-ipc/ipc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/samsung-ipc/ipc.c b/samsung-ipc/ipc.c index 993a3ab..62bce28 100644 --- a/samsung-ipc/ipc.c +++ b/samsung-ipc/ipc.c @@ -399,14 +399,14 @@ int ipc_client_gprs_deactivate(struct ipc_client *client) return client->handlers->gprs_deactivate(client->handlers->gprs_deactivate_data); } -int ipc_client_gprs_get_iface(struct ipc_client *client, char **iface, int cid) +char* ipc_client_gprs_get_iface(struct ipc_client *client, int cid) { if (client == NULL || client->gprs_specs == NULL || client->gprs_specs->gprs_get_iface == NULL) - return -1; + return NULL; - return client->gprs_specs->gprs_get_iface(iface, cid); + return client->gprs_specs->gprs_get_iface(cid); } int ipc_client_gprs_get_capabilities(struct ipc_client *client, struct ipc_client_gprs_capabilities *cap) |