Re: [PATCH 13/15] staging: usbip: userspace: libsrc: cleanup parsing

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

 



On Thu, Apr 04, 2013 at 10:10:10PM +0300, Dan Carpenter wrote:
> > +		if (buf[0] == 'P' && buf[1] == 'H' && buf[2] == 'Y' &&
> > +		    buf[3] == 'S' && buf[4] == 'D' &&
> > +		    buf[5] == 'E' && buf[6] == 'S' && /*isspace(buf[7])*/
> > +		    buf[7] == ' ') {
> >  			continue;
> 
> 	if (strncmp(buf, "PHYSDES ", 8) == 0)
> 		continue;

Btw, this is obviously something that could be cleaned up in a later
patch.  Generally, if you didn't introduce it and it's not a bug
then fixing it in a later patch is fine.

regards,
dan carpenter
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux