Re: [PATCH] Some code cleanup in hdp

From: Jose Antonio Santos Cadenas
Date: Wed Oct 13 2010 - 09:52:33 EST


Sorry, wrong mailing list.

2010/10/13 Jose Antonio Santos Cadenas <santoscadenas@xxxxxxxxx>:
> ---
>  health/hdp.c      |   38 ++++++++++++++++++++++----------------
>  health/hdp_util.c |   29 ++++++++++++++++++-----------
>  2 files changed, 40 insertions(+), 27 deletions(-)
>
> diff --git a/health/hdp.c b/health/hdp.c
> index 4a6da78..954b6d6 100644
> --- a/health/hdp.c
> +++ b/health/hdp.c
> @@ -305,11 +305,12 @@ static DBusMessage *manager_create_application(DBusConnection *conn,
>        const char *name;
>        DBusMessageIter iter;
>        GError *err = NULL;
> -       DBusMessage *reply;
>
>        dbus_message_iter_init(msg, &iter);
>        app = hdp_get_app_config(&iter, &err);
>        if (err) {
> +               DBusMessage *reply;
> +
>                reply = g_dbus_create_error(msg,
>                                        ERROR_INTERFACE ".InvalidArguments",
>                                        "Invalid arguments: %s", err->message);
> @@ -439,12 +440,13 @@ static void abort_mdl_cb(GError *err, gpointer data)
>  static void hdp_mdl_reconn_cb(struct mcap_mdl *mdl, GError *err, gpointer data)
>  {
>        struct hdp_tmp_dc_data *dc_data = data;
> -       struct hdp_channel *chan = dc_data->hdp_chann;
> -       GError *gerr = NULL;
>        DBusMessage *reply;
>        int fd;
>
>        if (err) {
> +               struct hdp_channel *chan = dc_data->hdp_chann;
> +               GError *gerr = NULL;
> +
>                error("%s", err->message);
>                reply = g_dbus_create_error(dc_data->msg,
>                                        ERROR_INTERFACE ".HealthError",
> @@ -633,12 +635,13 @@ static void health_channel_destroy(void *data)
>  {
>        struct hdp_channel *hdp_chan = data;
>        struct hdp_device *dev = hdp_chan->dev;
> -       char *empty_path;
>
>        DBG("Destroy Health Channel %s", hdp_chan->path);
>        dev->channels = g_slist_remove(dev->channels, hdp_chan);
>
>        if (hdp_chan == dev->fr) {
> +               char *empty_path;
> +
>                dev->fr = NULL;
>                empty_path = "";
>                emit_property_changed(dev->conn, device_get_path(dev->dev),
> @@ -754,8 +757,6 @@ static void hdp_mcap_mdl_connected_cb(struct mcap_mdl *mdl, void *data)
>  {
>        struct hdp_device *dev = data;
>        struct hdp_channel *chan;
> -       GIOChannel *io_chan;
> -       int fd;
>
>        DBG("hdp_mcap_mdl_connected_cb");
>        if (!dev->ndc)
> @@ -769,16 +770,19 @@ static void hdp_mcap_mdl_connected_cb(struct mcap_mdl *mdl, void *data)
>                dev->channels = g_slist_prepend(dev->channels, chan);
>
>        if (chan->mdep == HDP_MDEP_ECHO) {
> +               GIOChannel *io;
> +               int fd;
> +
>                fd = mcap_mdl_get_fd(chan->mdl);
>                if (fd < 0)
>                        return;
>
>                chan->echo_done = FALSE;
> -               io_chan = g_io_channel_unix_new(fd);
> -               chan->wid = g_io_add_watch(io_chan,
> +               io = g_io_channel_unix_new(fd);
> +               chan->wid = g_io_add_watch(io,
>                                G_IO_ERR | G_IO_HUP | G_IO_NVAL | G_IO_IN,
>                                serve_echo, chan);
> -               g_io_channel_unref(io_chan);
> +               g_io_channel_unref(io);
>                return;
>        }
>
> @@ -859,8 +863,6 @@ static uint8_t hdp_mcap_mdl_conn_req_cb(struct mcap_mcl *mcl, uint8_t mdepid,
>  {
>        struct hdp_device *dev = data;
>        struct hdp_application *app;
> -       struct hdp_channel *chan;
> -       char *path;
>        GSList *l;
>
>        DBG("Data channel request");
> @@ -922,7 +924,9 @@ static uint8_t hdp_mcap_mdl_conn_req_cb(struct mcap_mcl *mcl, uint8_t mdepid,
>
>        l = g_slist_find_custom(dev->channels, &mdlid, cmp_chan_mdlid);
>        if (l) {
> -               chan = l->data;
> +               struct hdp_channel *chan = l->data;
> +               char *path;
> +
>                path = g_strdup(chan->path);
>                g_dbus_unregister_interface(dev->conn, path, HEALTH_CHANNEL);
>                g_free(path);
> @@ -981,16 +985,17 @@ gboolean hdp_set_mcl_cb(struct hdp_device *device, GError **err)
>
>  static void mcl_connected(struct mcap_mcl *mcl, gpointer data)
>  {
> -       struct hdp_adapter *hdp_adapter = data;
>        struct hdp_device *hdp_device;
> -       struct btd_device *device;
>        bdaddr_t addr;
> -       char str[18];
>        GSList *l;
>
>        mcap_mcl_get_addr(mcl, &addr);
>        l = g_slist_find_custom(devices, &addr, cmp_dev_addr);
>        if (!l) {
> +               struct hdp_adapter *hdp_adapter = data;
> +               struct btd_device *device;
> +               char str[18];
> +
>                ba2str(&addr, str);
>                device = adapter_get_device(connection,
>                                        hdp_adapter->btd_adapter, str);
> @@ -1088,7 +1093,6 @@ static void check_devices_mcl()
>  {
>        struct hdp_device *dev;
>        GSList *l, *to_delete = NULL;
> -       const char *path;
>
>        for (l = devices; l; l = l->next) {
>                dev = l->data;
> @@ -1101,6 +1105,8 @@ static void check_devices_mcl()
>        }
>
>        for (l = to_delete; l; l = l->next) {
> +               const char *path;
> +
>                path = device_get_path(dev->dev);
>                g_dbus_unregister_interface(dev->conn, path, HEALTH_DEVICE);
>        }
> diff --git a/health/hdp_util.c b/health/hdp_util.c
> index ef76b15..c5dfb01 100644
> --- a/health/hdp_util.c
> +++ b/health/hdp_util.c
> @@ -141,12 +141,14 @@ static gboolean parse_data_type(DBusMessageIter *iter, gpointer data,
>                                                                GError **err)
>  {
>        struct hdp_application *app = data;
> -       DBusMessageIter *value, variant;
> +       DBusMessageIter *value;
>        int ctype;
>
>        ctype = dbus_message_iter_get_arg_type(iter);
>        value = iter;
>        if (ctype == DBUS_TYPE_VARIANT) {
> +               DBusMessageIter variant;
> +
>                /* Get value inside the variable */
>                dbus_message_iter_recurse(iter, &variant);
>                ctype = dbus_message_iter_get_arg_type(&variant);
> @@ -167,13 +169,14 @@ static gboolean parse_data_type(DBusMessageIter *iter, gpointer data,
>  static gboolean parse_role(DBusMessageIter *iter, gpointer data, GError **err)
>  {
>        struct hdp_application *app = data;
> -       DBusMessageIter value;
>        DBusMessageIter *string;
>        int ctype;
>        const char *role;
>
>        ctype = dbus_message_iter_get_arg_type(iter);
>        if (ctype == DBUS_TYPE_VARIANT) {
> +               DBusMessageIter value;
> +
>                /* Get value inside the variable */
>                dbus_message_iter_recurse(iter, &value);
>                ctype = dbus_message_iter_get_arg_type(&value);
> @@ -206,12 +209,14 @@ static gboolean parse_role(DBusMessageIter *iter, gpointer data, GError **err)
>  static gboolean parse_desc(DBusMessageIter *iter, gpointer data, GError **err)
>  {
>        struct hdp_application *app = data;
> -       DBusMessageIter *string, variant;
> +       DBusMessageIter *string;
>        int ctype;
>        const char *desc;
>
>        ctype = dbus_message_iter_get_arg_type(iter);
>        if (ctype == DBUS_TYPE_VARIANT) {
> +               DBusMessageIter variant;
> +
>                /* Get value inside the variable */
>                dbus_message_iter_recurse(iter, &variant);
>                ctype = dbus_message_iter_get_arg_type(&variant);
> @@ -234,12 +239,14 @@ static gboolean parse_chan_type(DBusMessageIter *iter, gpointer data,
>                                                                GError **err)
>  {
>        struct hdp_application *app = data;
> -       DBusMessageIter *value, variant;
> +       DBusMessageIter *value;
>        int ctype;
>
>        ctype = dbus_message_iter_get_arg_type(iter);
>        value = iter;
>        if (ctype == DBUS_TYPE_VARIANT) {
> +               DBusMessageIter variant;
> +
>                /* Get value inside the variable */
>                dbus_message_iter_recurse(iter, &variant);
>                ctype = dbus_message_iter_get_arg_type(&variant);
> @@ -294,11 +301,11 @@ fail:
>
>  static gboolean is_app_role(GSList *app_list, HdpRole role)
>  {
> -       struct hdp_application *app;
>        GSList *l;
>
>        for (l = app_list; l; l = l->next) {
> -               app = l->data;
> +               struct hdp_application *app = l->data;
> +
>                if (app->role == role)
>                        return TRUE;
>        }
> @@ -732,7 +739,7 @@ static gboolean check_role(uint8_t rec_role, uint8_t app_role)
>  static gboolean get_mdep_from_rec(const sdp_record_t *rec, uint8_t role,
>                                uint16_t d_type, uint8_t *mdep, char **desc)
>  {
> -       sdp_data_t *list, *feat, *data_type, *mdepid, *role_t, *desc_t;
> +       sdp_data_t *list, *feat;
>
>        if (!desc && !mdep)
>                return TRUE;
> @@ -744,6 +751,8 @@ static gboolean get_mdep_from_rec(const sdp_record_t *rec, uint8_t role,
>                return FALSE;
>
>        for (feat = list->val.dataseq; feat; feat = feat->next) {
> +               sdp_data_t *data_type, *mdepid, *role_t, *desc_t;
> +
>                if (feat->dtd != SDP_SEQ8 && feat->dtd != SDP_SEQ16 &&
>                                                feat->dtd != SDP_SEQ32)
>                        continue;
> @@ -931,10 +940,9 @@ static gboolean hdp_get_add_prot_desc_list(const sdp_record_t *rec,
>  static gboolean get_ccpsm(sdp_list_t *recs, uint16_t *ccpsm)
>  {
>        sdp_list_t *l;
> -       sdp_record_t *rec;
>
>        for (l = recs; l; l = l->next) {
> -               rec = l->data;
> +               sdp_record_t *rec = l->data;
>
>                if (hdp_get_prot_desc_list(rec, ccpsm, NULL))
>                        return TRUE;
> @@ -946,10 +954,9 @@ static gboolean get_ccpsm(sdp_list_t *recs, uint16_t *ccpsm)
>  static gboolean get_dcpsm(sdp_list_t *recs, uint16_t *dcpsm)
>  {
>        sdp_list_t *l;
> -       sdp_record_t *rec;
>
>        for (l = recs; l; l = l->next) {
> -               rec = l->data;
> +               sdp_record_t *rec = l->data;
>
>                if (hdp_get_add_prot_desc_list(rec, dcpsm))
>                        return TRUE;
> --
> 1.7.1
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/