Skip to content

fprintf(stderr, "warning: ") -> warning() patches

Miklos Vajna git 1 version Applied

Status from this message: Applied.

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 6 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/9http-push: using error() and warning() as appropriatehttp-push.c+3 -30 replies
2/9builtin-apply: use warning() instead of fprintf(stderr, "warning: ")builtin-apply.c+7 -81 reply
3/9builtin-checkout: use warning() instead of fprintf(stderr, "warning: ")builtin-checkout.c+2 -20 replies
4/9builtin-fetch-pack: use warning() instead of fprintf(stderr, "warning: ")builtin-fetch-pack.c+1 -10 replies
5/9builtin-init-db: use warning() instead of fprintf(stderr, "warning: ")builtin-init-db.c+3 -40 replies
6/9builtin-rm: use warning() instead of fprintf(stderr, "warning: ")builtin-rm.c+1 -20 replies
7/9builtin-show-branch: use warning() instead of fprintf(stderr, "warning: ")builtin-show-branch.c+1 -20 replies
8/9builtin-show-ref: use warning() instead of fprintf(stderr, "warning: ")builtin-show-ref.c+1 -10 replies
9/9refs: use warning() instead of fprintf(stderr, "warning: ")refs.c+3 -53 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/9] fprintf(stderr, "warning: ") -> warning() patches
Quoted 3 lines · Junio C Hamano
Quoted 1 line

Should I rebase the series against current master and resend?

Yes, please, if the overlap with "diff master next" is small enough.

Okay, here it is, applies on top of master. All, except the http-push one, applies cleanly on top of next as well here.

Miklos Vajna (9):
http-push: using error() and warning() as appropriate
builtin-apply: use warning() instead of fprintf(stderr, "warning: ")
builtin-checkout: use warning() instead of fprintf(stderr, "warning:

    ")
  builtin-fetch-pack: use warning() instead of fprintf(stderr,
    "warning: ")
  builtin-init-db: use warning() instead of fprintf(stderr, "warning:
    ")
  builtin-rm: use warning() instead of fprintf(stderr, "warning: ")
  builtin-show-branch: use warning() instead of fprintf(stderr,
    "warning: ")
  builtin-show-ref: use warning() instead of fprintf(stderr, "warning:
    ")

refs: use warning() instead of fprintf(stderr, "warning: ")

 builtin-apply.c       |   15 +++++++--------
 builtin-checkout.c    |    4 ++--
 builtin-fetch-pack.c  |    2 +-
 builtin-init-db.c     |    7 +++----
 builtin-rm.c          |    3 +--
 builtin-show-branch.c |    3 +--
 builtin-show-ref.c    |    2 +-
 http-push.c           |    6 +++---
 refs.c                |    8 +++-----
 9 files changed, 22 insertions(+), 28 deletions(-)