From: Matthieu Moy <Matthieu.Moy@imag.fr>
To: git@vger.kernel.org, gitster@pobox.com
Cc: Matthieu Moy <Matthieu.Moy@imag.fr>
Subject: [PATCH v5 6/6] status: add missing blank line after list of "other" files
Date: Fri, 6 Sep 2013 19:43:09 +0200 [thread overview]
Message-ID: <1378489389-25802-7-git-send-email-Matthieu.Moy@imag.fr> (raw)
In-Reply-To: <1378489389-25802-1-git-send-email-Matthieu.Moy@imag.fr>
List of files in other sections ("Changes to be committed", ...) end with
a blank line. It is not the case with the "Untracked files" and "Ignored
files" sections. The issue become particularly visible after the #-prefix
removal, as the last line (e.g. "nothing added to commit but untracked
files present") seems mixed with the untracked files.
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
---
t/t7508-status.sh | 21 +++++++++++++++++++++
wt-status.c | 4 +++-
2 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/t/t7508-status.sh b/t/t7508-status.sh
index d0444d3..9bf9701 100755
--- a/t/t7508-status.sh
+++ b/t/t7508-status.sh
@@ -84,6 +84,7 @@ test_expect_success 'status --column' '
#
# dir1/untracked dir2/untracked output
# dir2/modified expect untracked
+#
EOF
COLUMNS=50 git -c status.displayCommentPrefix=true status --column="column dense" >output &&
test_i18ncmp expect output
@@ -117,6 +118,7 @@ cat >expect <<\EOF
# expect
# output
# untracked
+#
EOF
test_expect_success 'status with status.displayCommentPrefix=true' '
@@ -167,6 +169,7 @@ Untracked files:
expect
output
untracked
+
EOF
test_expect_success 'status (advice.statusHints false)' '
@@ -241,6 +244,7 @@ Untracked files:
(use "git add <file>..." to include in what will be committed)
dir2/modified
+
Ignored files:
(use "git add -f <file>..." to include in what will be committed)
@@ -250,6 +254,7 @@ Ignored files:
expect
output
untracked
+
EOF
git status --ignored >output &&
test_i18ncmp expect output
@@ -308,6 +313,7 @@ Ignored files:
expect
output
untracked
+
EOF
git status --ignored >output &&
test_i18ncmp expect output
@@ -430,6 +436,7 @@ Untracked files:
expect
output
untracked
+
EOF
git status -unormal >output &&
test_i18ncmp expect output
@@ -488,6 +495,7 @@ Untracked files:
expect
output
untracked
+
EOF
git status -uall >output &&
test_i18ncmp expect output
@@ -548,6 +556,7 @@ Untracked files:
../expect
../output
../untracked
+
EOF
(cd dir1 && git status) >output &&
test_i18ncmp expect output
@@ -618,6 +627,7 @@ Untracked files:
<BLUE>expect<RESET>
<BLUE>output<RESET>
<BLUE>untracked<RESET>
+
EOF
test_config color.ui always &&
git status | test_decode_color >output &&
@@ -747,6 +757,7 @@ Untracked files:
expect
output
untracked
+
EOF
test_config status.relativePaths false &&
(cd dir1 && git status) >output &&
@@ -789,6 +800,7 @@ Untracked files:
expect
output
untracked
+
EOF
git commit --dry-run dir1/modified >output &&
test_i18ncmp expect output
@@ -838,6 +850,7 @@ Untracked files:
expect
output
untracked
+
EOF
git status >output &&
test_i18ncmp expect output
@@ -902,6 +915,7 @@ Untracked files:
expect
output
untracked
+
EOF
git config status.submodulesummary 10 &&
git status >output &&
@@ -952,6 +966,7 @@ Untracked files:
expect
output
untracked
+
no changes added to commit (use "git add" and/or "git commit -a")
EOF
git commit -m "commit submodule" &&
@@ -1012,6 +1027,7 @@ Untracked files:
expect
output
untracked
+
EOF
git config status.submodulesummary 10 &&
git commit --dry-run --amend >output &&
@@ -1066,6 +1082,7 @@ Untracked files:
expect
output
untracked
+
EOF
echo modified sm/untracked &&
git status --ignore-submodules=untracked >output &&
@@ -1177,6 +1194,7 @@ Untracked files:
expect
output
untracked
+
EOF
git status --ignore-submodules=untracked > output &&
test_i18ncmp expect output
@@ -1238,6 +1256,7 @@ Untracked files:
expect
output
untracked
+
EOF
git status --ignore-submodules=untracked > output &&
test_i18ncmp expect output
@@ -1319,6 +1338,7 @@ cat > expect << EOF
; expect
; output
; untracked
+;
EOF
test_expect_success "status (core.commentchar with submodule summary)" '
@@ -1352,6 +1372,7 @@ Untracked files:
expect
output
untracked
+
no changes added to commit (use "git add" and/or "git commit -a")
EOF
git status --ignore-submodules=all > output &&
diff --git a/wt-status.c b/wt-status.c
index 3c795da..2a9ca0f 100644
--- a/wt-status.c
+++ b/wt-status.c
@@ -751,7 +751,7 @@ static void wt_status_print_other(struct wt_status *s,
strbuf_release(&buf);
if (!column_active(s->colopts))
- return;
+ goto conclude;
strbuf_addf(&buf, "%s%s\t%s",
color(WT_STATUS_HEADER, s),
@@ -765,6 +765,8 @@ static void wt_status_print_other(struct wt_status *s,
print_columns(&output, s->colopts, &copts);
string_list_clear(&output, 0);
strbuf_release(&buf);
+conclude:
+ status_printf_ln(s, GIT_COLOR_NORMAL, "");
}
static void wt_status_print_verbose(struct wt_status *s)
--
1.8.4.5.g8688bea
next prev parent reply other threads:[~2013-09-06 17:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <vpq38phsvuc.fsf@anie.imag.frvpq38phsvuc.fsf@anie.imag.fr>
2013-09-06 17:43 ` [PATCH v5 0/6] Disable "git status" comment prefix Matthieu Moy
2013-09-06 17:43 ` [PATCH v5 1/6] builtin/stripspace.c: fix broken indentation Matthieu Moy
2013-09-06 17:43 ` [PATCH v5 2/6] wt-status: use argv_array API Matthieu Moy
2013-09-06 17:43 ` [PATCH v5 3/6] submodule summary: ignore --for-status option Matthieu Moy
2013-09-06 17:43 ` [PATCH v5 4/6] status: disable display of '#' comment prefix by default Matthieu Moy
2013-09-06 17:43 ` [PATCH v5 5/6] tests: don't set status.displayCommentPrefix file-wide Matthieu Moy
2013-09-06 17:43 ` Matthieu Moy [this message]
2013-09-06 18:22 ` [PATCH v5 6/6] status: add missing blank line after list of "other" files Junio C Hamano
2013-09-06 18:42 ` Junio C Hamano
2013-09-06 22:09 ` Jeff King
2013-09-06 18:45 ` Matthieu Moy
2013-09-06 18:56 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1378489389-25802-7-git-send-email-Matthieu.Moy@imag.fr \
--to=matthieu.moy@imag.fr \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).