[PATCH] option,hso: minor coding style fixes

Aleksander Morgado aleksander at aleksander.es
Tue May 20 00:34:51 PDT 2014


On 20/05/14 08:45, Ben Chan wrote:
> ---
>  plugins/option/mm-broadband-bearer-hso.c   | 16 ++++++++--------
>  plugins/option/mm-broadband-modem-hso.c    |  2 +-
>  plugins/option/mm-broadband-modem-option.c |  2 +-
>  3 files changed, 10 insertions(+), 10 deletions(-)
> 

Pushed, thanks.

> diff --git a/plugins/option/mm-broadband-bearer-hso.c b/plugins/option/mm-broadband-bearer-hso.c
> index 0f361f4..4b128e5 100644
> --- a/plugins/option/mm-broadband-bearer-hso.c
> +++ b/plugins/option/mm-broadband-bearer-hso.c
> @@ -568,12 +568,12 @@ authenticate_ready (MMBaseModem *modem,
>  }
>  
>  const gchar *auth_commands[] = {
> -	"$QCPDPP",
> -	/* Icera-based devices (GI0322/Quicksilver, iCON 505) don't implement
> -	 * $QCPDPP, but instead use _OPDPP with the same arguments.
> -	 */
> -	"_OPDPP",
> -	NULL
> +    "$QCPDPP",
> +    /* Icera-based devices (GI0322/Quicksilver, iCON 505) don't implement
> +     * $QCPDPP, but instead use _OPDPP with the same arguments.
> +     */
> +    "_OPDPP",
> +    NULL
>  };
>  
>  static void
> @@ -600,7 +600,7 @@ authenticate (Dial3gppContext *ctx)
>      /* Both user and password are required; otherwise firmware returns an error */
>      if (!user || !password || allowed_auth == MM_BEARER_ALLOWED_AUTH_NONE) {
>          mm_dbg ("Not using authentication");
> -		command = g_strdup_printf ("%s=%d,0",
> +        command = g_strdup_printf ("%s=%d,0",
>                                     auth_commands[ctx->auth_idx],
>                                     ctx->cid);
>      } else {
> @@ -828,7 +828,7 @@ static void
>  mm_broadband_bearer_hso_init (MMBroadbandBearerHso *self)
>  {
>      /* Initialize private data */
> -    self->priv = G_TYPE_INSTANCE_GET_PRIVATE ((self),
> +    self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self,
>                                                MM_TYPE_BROADBAND_BEARER_HSO,
>                                                MMBroadbandBearerHsoPrivate);
>  }
> diff --git a/plugins/option/mm-broadband-modem-hso.c b/plugins/option/mm-broadband-modem-hso.c
> index fd3df34..a189959 100644
> --- a/plugins/option/mm-broadband-modem-hso.c
> +++ b/plugins/option/mm-broadband-modem-hso.c
> @@ -755,7 +755,7 @@ static void
>  mm_broadband_modem_hso_init (MMBroadbandModemHso *self)
>  {
>      /* Initialize private data */
> -    self->priv = G_TYPE_INSTANCE_GET_PRIVATE ((self),
> +    self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self,
>                                                MM_TYPE_BROADBAND_MODEM_HSO,
>                                                MMBroadbandModemHsoPrivate);
>  
> diff --git a/plugins/option/mm-broadband-modem-option.c b/plugins/option/mm-broadband-modem-option.c
> index c8409dc..cf02ba3 100644
> --- a/plugins/option/mm-broadband-modem-option.c
> +++ b/plugins/option/mm-broadband-modem-option.c
> @@ -1203,7 +1203,7 @@ static void
>  mm_broadband_modem_option_init (MMBroadbandModemOption *self)
>  {
>      /* Initialize private data */
> -    self->priv = G_TYPE_INSTANCE_GET_PRIVATE ((self),
> +    self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self,
>                                                MM_TYPE_BROADBAND_MODEM_OPTION,
>                                                MMBroadbandModemOptionPrivate);
>      self->priv->after_power_up_wait_id = 0;
> 


-- 
Aleksander
https://aleksander.es


More information about the ModemManager-devel mailing list