You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes, completely sound to omit them for a single command.
There are times you really, really want to do it anyway, though: notably, at function definitions. (POSIX says f() echo hi is a perfectly good definition of a function f, but e.g., bash rejects it.)
I feel like the real thing we want here is a properly tested, properly written pretty printer. The tests on the OCaml AST from libdash are obviously insufficient, and the printer itself is wrong in numerous places.
Currently (after the recent changes to improve posix compliance) there are too many braces in the output script.
For example, here is the mkfifo function for
evaluation/intro/demo-spell.sh
. 馃あOnce we have a posix compliance infrastructure setup, we can play around with different ways to print less verbose code without breaking correctness.
@mgree do you have any thoughts on the above? For example, would it be sound to not add braces around single commands?
The text was updated successfully, but these errors were encountered: