Hi Martijn,
Martijn van Duren wrote on Thu, Dec 06, 2018 at 09:12:19PM +0100:
> OK martijn@
Thanks for checking!
After testing that "make build" and "make release" still works,
i just committed the patch.
Yours,
Ingo
CVSROOT: /cvs
Module name: src
Changes by: schwa...@cvs.openbsd.org 2018/12/07 07:45:40
Modified files:
usr.bin/sed : compile.c
Log message:
As an extension to POSIX, for consistency with our behaviour for
the "b" and "t" commands with a label, and for compatibility with
GNU sed, also accept ";" followed by another command after "b"
and "t" commands without a label: branch to the end of the script
instead of erroring out. Parsing is unchanged.
Missing feature reported by Lars dot Nooden at gmail dot com on bugs@.
OK martijn@ millert@
>> Index: compile.c
>> ===================================================================
>> RCS file: /cvs/src/usr.bin/sed/compile.c,v
>> retrieving revision 1.49
>> diff -u -r1.49 compile.c
>> --- compile.c 14 Aug 2018 18:10:09 -0000 1.49
>> +++ compile.c 6 Dec 2018 19:07:31 -0000
>> @@ -284,7 +284,7 @@
>> case BRANCH: /* b t */
>> p++;
>> EATSPACE();
>> - if (*p == '\0')
>> + if (*p == '\0' || *p == ';')
>> cmd->t = NULL;
>> else
>> cmd->t = duptoeol(p, "branch", &p);
>>
