Re: [PATCH v8 36/41] apply: don't print on stdout when be_silent is set
From: Junio C Hamano <hidden>
Date: 2016-07-26 19:41:47
Christian Couder [off-list ref] writes:
This variable should prevent anything to be printed on both stderr and stdout.
You have to mention that skipping the entire callchain, not just the "printing" part, is safe. I can see numstat_patch_list() is probably safe as it does not do any computation other than calling printf() and write_name_quoted(), but other two are not immediately obvious that what they compute are only used for their own printing and there is no other side effects left to affect what happens after this function returns.
quoted hunk
Signed-off-by: Christian Couder <redacted> --- apply.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)diff --git a/apply.c b/apply.c index 1435f85..e2acc18 100644 --- a/apply.c +++ b/apply.c@@ -4698,13 +4698,13 @@ static int apply_patch(struct apply_state *state, goto end; } - if (state->diffstat) + if (state->diffstat && !state->be_silent) stat_patch_list(state, list); - if (state->numstat) + if (state->numstat && !state->be_silent) numstat_patch_list(state, list); - if (state->summary) + if (state->summary && !state->be_silent) summary_patch_list(list); end: