On Thu, Aug 14, 2025 at 03:17:15PM -0700, Marc Herbert wrote:

snip
> 
> If nothing else, the comment should be a bit more specific and mention
> the relevant "nameref" or "indirection" keyword.

Done - used both keywords in a more verbose explanation in v2.

snip
> >>> +        if [ "$expect_failures" = "false" ]; then
> >>
> >>        if "$expect_failures"; then
> > 
> > Is the existing pattern wrong or is that an ask for brevity?
> 
> Yes, just for brevity and clarity. I mean you would not write "if
> (some_bool == false)" in another language. Not important.

Disconnect is I was thinking string compares, not bools. I changed
this in v2 to use builtin true/false bash-isms

Thanks for the reviews!

Reply via email to