diff options
| author | Luke Shumaker <lukeshu@datawire.io> | 2021-04-27 15:17:30 -0600 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2021-04-28 16:47:17 +0900 |
| commit | 5a3569774f7b6dcdbdf470ae1e40367200d987ec (patch) | |
| tree | b75f921ff9c774049433b547ae670bfb1cab292b /t/t4018/cpp-function-returning-pointer | |
| parent | subtree: t7900: add porcelain tests for 'pull' and 'push' (diff) | |
| download | git-5a3569774f7b6dcdbdf470ae1e40367200d987ec.tar.gz git-5a3569774f7b6dcdbdf470ae1e40367200d987ec.zip | |
subtree: don't have loose code outside of a function
Shove all of the loose code inside of a main() function.
This comes down to personal preference more than anything else. A
preference that I've developed over years of maintaining large Bash
scripts, but still a mere personal preference.
In this specific case, it's also moving the `set -- -h`, the `git
rev-parse --parseopt`, and the `. git-sh-setup` to be closer to all
the rest of the argument parsing, which is a readability win on its
own, IMO.
"Ignore space change" is probably helpful when viewing this diff.
Signed-off-by: Luke Shumaker <lukeshu@datawire.io>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t4018/cpp-function-returning-pointer')
0 files changed, 0 insertions, 0 deletions