Conversation
Added spaces to make the trailing line-continuation slashes all line up in the same column. Also moved a few command-line flags around to group together flags of same kind.
|
Here's the URL to review the code without the whitespace diff https://github.com/containers/bubblewrap/pull/394/files?diff=unified&w=1 |
|
Can one of the admins verify this patch?
|
|
Reviving the old PR. @alexlarsson , others, please take a look at the minimal diff of rearranging lines; consider ignoring the whitespace diff during review by using the following link. https://github.com/containers/bubblewrap/pull/394/files?diff=unified&w=1 |
I personally think this doesn't make it any prettier, and the need to redo the lining-up every time a longer parameter is added is a practical annoyance. |
This does maybe make sense, although the order of parameters to |
Added spaces to make the trailing line-continuation slashes all line up
in the same column. Also moved a few command-line flags around to group
together flags of same kind.
Note to reviewers: use command
git diff -wto see the changes to excludethe whitespace diff.