Re: dash bug: double-quoted "\" breaks glob protection for next char

2018-03-09 Thread Harald van Dijk
On 3/8/18 1:40 AM, Harald van Dijk wrote: If the syntax stack is to be stored on the actual stack, then real recursion could be used instead, as attached. Even though it won't be accepted in dash, I continued with this approach for my own use. I've now got it to about 1800 bytes smaller (at

Re: [PATCH] parser: Fix single-quoted patterns in here-documents

2018-03-09 Thread Harald van Dijk
On 3/9/18 4:07 PM, Herbert Xu wrote: On Thu, Mar 08, 2018 at 07:35:53PM +0100, Harald van Dijk wrote: Related: x=*; cat < I don't think this is related to our patches at all. Not related to our patches, but related to the original bug. It's another instance where quoted * is wrongly

[PATCH] parser: Fix single-quoted patterns in here-documents

2018-03-09 Thread Herbert Xu
On Thu, Mar 08, 2018 at 07:35:53PM +0100, Harald van Dijk wrote: > > Related: > > x=*; cat < ${x#'*'} > EOF > > This shouldn't print anything either: because the * is quoted, it should be > taken as a literal and removed from $x. > > Re-testing, I see this didn't work properly with my