3 ms·
The clang-format wrapper script presented has a bug. The $@ at the end must be quoted for it to work on filenames containing spaces. For bonus points it should
by jzwinck 11y ago
The clang-format wrapper script presented has a bug. The $@ at the end must be quoted for it to work on filenames containing spaces.
For bonus points it shouldd prefix the clang-format command with "exec" but that's not a correctness issue.
- minitech 11y agoDo you mean “must not be quoted”? $@ shouldn’t be, most of the time.
- jzwinck 11y agoNo typo. It must be quoted as "$@". If you think otherwise, test with some filenames containing spaces. Or read here: http://stackoverflow.com/questions/448407/bash-script-to-receive-and-repass-quoted-parameters http://stackoverflow.com/questions/448407/bash-script-to-rec...