(1) Move the stripspace() function to the strbuf module adding a prefix
and changing all users accordingly. Also introduce a wrapper in case
any topic branches still depend on the old name.
(2) Switch git stripspace to use parse-options in order to simplify
introducing new command line options (as in the following patch). In
v1 this was folded into patch (3) and is now split out for v2.
(3) Introduce option --count-lines to git stripspace and add the
corresponding documentation and tests.
(4) Change git-rebase--interactive.sh to replace commands like:
git stripspace ... | wc -l
with:
git stripspace --count-lines ...
This patch set implements some of the project ideas around git stripspace
suggested on https://git.wiki.kernel.org/index.php/SmallProjectsIdeas
v1 -> v2:
- Thanks to Junio and Matthieu for the review.
- Split patch 2/3 into two patches: patch 2/4 switches git stripspace
to use parse-options and patch 3/4 introduces the new option.
- Implement line counting in cmd_stripbuf() instead of (ab-)using
strbuf_stripspace() for it.
- Drop -C short option
- Correct example command output in documentation.
- Adjust commit messages to not include links to the wiki, fully
describe the motivation in the commit message instead.
Tobias Klauser (4):
strbuf: make stripspace() part of strbuf
stripspace: Use parse-options for command-line parsing
stripspace: Implement --count-lines option
git rebase -i: Use newly added --count-lines option for stripspace
Documentation/git-stripspace.txt | 14 +++-
builtin/am.c | 2 +-
builtin/branch.c | 2 +-
builtin/commit.c | 6 +-
builtin/merge.c | 2 +-
builtin/notes.c | 6 +-
builtin/stripspace.c | 137 +++++++++++++--------------------------
builtin/tag.c | 2 +-
git-rebase--interactive.sh | 6 +-
strbuf.c | 66 +++++++++++++++++++
strbuf.h | 11 +++-
t/t0030-stripspace.sh | 36 ++++++++++
12 files changed, 181 insertions(+), 109 deletions(-)
--
2.6.1.148.g7927db1
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
