RE: [PATCH] tools/seq2bseq : Fix the same expression issue in if condition

[Gowtham Anandha Babu <gowtham.ab@samsung.com>]

Hi ,

> -----Original Message-----
> From: Marcel Holtmann [mailto:marcel@holtmann.org]
> Sent: Friday, September 12, 2014 10:35 PM
> To: Johan Hedberg
> Cc: gowtham babu; linux-bluetooth@vger.kernel.org;
> d.kasatkin@samsung.com; bharat.panda@samsung.com;
> cpgs@samsung.com
> Subject: Re: [PATCH] tools/seq2bseq : Fix the same expression issue in if
> condition
> 
> Hi Johan,
> 
> >> Below are the list of similar patches under 'tools' which are not yet
> >> reviewed
> >>
> >> 1) tools/csr_usb : Fix Resource leak: file
> >> 2) tools/hciattach : Fix syntax error
> >> 3) tools/btsnoop : Fix variable reassigning issue
> >
> > This kind of stuff does not belong in a commit message.
> >
> >> diff --git a/tools/seq2bseq.c b/tools/seq2bseq.c index
> >> 7657a57..521d20e 100644
> >> --- a/tools/seq2bseq.c
> >> +++ b/tools/seq2bseq.c
> >> @@ -40,7 +40,7 @@ static int convert_line(int fd, const char *line)
> >> 	char str[3];
> >> 	unsigned char val;
> >>
> >> -	if (line[0] == '*' || line[0] == '\r' || line[0] == '\r')
> >> +	if (line[0] == '*' || line[0] == '\r')
> >> 		return 0;
> >>
> >> 	while (1) {
> >
> > Are you sure the code wasn't intending to check for '\n'?
> 
> yes, it was intended to check for \r and \n. This is my mistake here.
> 
> Regards
> 
> Marcel

I had fixed this and sent the revised patch. Please have a look at it. 

Regards,
Gowtham