Re: [PATCH] WIP: quote_string_for_globbing: unquoted backslash

2023-10-07 Thread Chet Ramey
On 9/26/23 2:50 AM, Grisha Levit wrote: I'm not confident in what the right behavior is here, and maybe there is no obvious one, but I _think_ this is not desirable: If an unquoted backslash is followed by a quoted globbing character, quote_string_for_globbing will store the unquoted backslash a

Re: PATCH] unquoted_glob_pattern_p: better recognition

2023-10-07 Thread Chet Ramey
On 9/26/23 2:25 AM, Grisha Levit wrote: A `/' followed by `(' should not be considered an unquoted glob character: $ bash -O failglob -c ': $1' _ 'a/(b)' _: line 1: no match: a/(b) If extglob is not turned on, it probably doesn't make sense to treat words that don't have any non-extended glob c

Re: [PATCH] glob_char_p: include extglob group chars

2023-10-07 Thread Chet Ramey
On 9/26/23 2:01 AM, Grisha Levit wrote: This was an oversight when changing quote_string_for_globbing to only quote chars when they need it. Thanks for the report. -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa, vita brevis'' - Hippocrates Chet Rame