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
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux