Re: [PATCH BlueZ 5/7] Add enum for alert level values

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Johan,

On Wed, Aug 3, 2011 at 3:41 PM, Claudio Takahasi
<claudio.takahasi@xxxxxxxxxxxxx> wrote:
> From: Bruna Moreira <bruna.moreira@xxxxxxxxxxxxx>
>
> ---
>  proximity/monitor.c |   12 +++++++++---
>  1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/proximity/monitor.c b/proximity/monitor.c
> index 6470db4..ca19a8f 100644
> --- a/proximity/monitor.c
> +++ b/proximity/monitor.c
> @@ -53,6 +53,12 @@
>
>  #define ALERT_LEVEL_CHR_UUID 0x2A06
>
> +enum {
> +       ALERT_NONE = 0,
> +       ALERT_MILD,
> +       ALERT_HIGH,
> +};
> +
>  struct monitor {
>        struct btd_device *device;
>        GAttrib *attrib;
> @@ -116,11 +122,11 @@ static char *read_proximity_config(bdaddr_t *sba, bdaddr_t *dba,
>  static uint8_t str2level(const char *level)
>  {
>        if (g_strcmp0("high", level) == 0)
> -               return 0x02;
> +               return ALERT_HIGH;
>        else if (g_strcmp0("mild", level) == 0)
> -               return 0x01;
> +               return ALERT_MILD;
>
> -       return 0x00;
> +       return ALERT_NONE;;

I gonna send a -v2 patch fixing this extra semicolon.

BR,
Claudio
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux