[Toybox] [PATCH] xargs: fix -E.

enh enh at google.com
Mon Oct 21 14:57:21 PDT 2019


yeah, passes the test. new patch attached with just the regression test.


On Mon, Oct 21, 2019 at 2:53 PM Rob Landley <rob at landley.net> wrote:
>
> On 10/18/19 7:22 PM, enh via Toybox wrote:
> > 89a8d00e470f1999a62ceea81269af2f39c655ba broke -E by switching to a
> > prefix match. Add the missing test and fix the bug, but caching the
> > length of the -E argument because the call to strlen(3) on every single
> > piece of input presumably motivated the change that introduced the bug.
>
> I fixed it a different way as part of a cleanup over the weekend, did that suffice?
>
> (Sorry, I've gotten into a "too many pending bug reports" swap-thrashing thing,
> I cut a release and walked away from the computer for a day or two...)
>
> Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-xargs-add-test-for-E-being-a-full-match.patch
Type: text/x-patch
Size: 1048 bytes
Desc: not available
URL: <http://lists.landley.net/pipermail/toybox-landley.net/attachments/20191021/1d61853d/attachment-0003.bin>


More information about the Toybox mailing list