On Fri, 2014-01-17 at 09:58 +0000, David Laight wrote:
> From: Sarah Sharp
> > > >- xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed
> > > >because "
> > > >- "slot %u was not enabled.\n",
> > > >slot_id);
> > > >+ xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed
> > > >because slot "
> > > >+ "%u was not enabled.\n",
> > > >slot_id);
>
> Why leave the line split at the %?
Because you can strictly speaking only guess that there's a '%'.
> I will grep for things like '".*because slot %'.
> Best to only split at an embedded \n.
Sorry, the idea just never occured to me. I'd have never dreamt of doing
such a grep.
Regards
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
