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