Hi Olivier, On 02/10/2011 07:59 AM, Olivier Guiter wrote: > --- > gatchat/gatserver.c | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > @@ -837,7 +838,11 @@ static char *extract_line(GAtServer *p, struct ring_buffer *rbuf) > if (*buf == '"') > in_string = !in_string; > > - if ((*buf == ' ' || *buf == '\t') && in_string == FALSE) > + if (*buf == s5) { > + if (i != 0) > + i -= 1; > + } > + else if ((*buf == ' ' || *buf == '\t') && in_string == FALSE) Please note that we follow the Linux kernel coding style, and in fact the closing brace should be on the same line as the else if. See CodingStyle document in the linux kernel git for more details. I've fixed this for you. Patch has been applied, thanks. Regards, -Denis