Skip to content

more warnings and cleanups

Jonathan Nieder git 1 version In review

Get series with b4
b4 am 20110316065256.GA5988@elie

Versions newest first

  1. v1 Latest8 patches 8 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/8enums: omit trailing comma for portabilitycache.h convert.c+2 -20 replies
2/8compat: make gcc bswap an inline functioncompat/bswap.h+10 -83 replies
3/8svn-fe: do not use "return" for tail call returning voidvcs-svn/svndump.c+2 -10 replies
4/8vcs-svn: remove spurious semicolonsvcs-svn/repo_tree.c vcs-svn/string_pool.c+2 -22 replies
5/8standardize brace placement in struct definitionsbuiltin/add.c builtin/blame.c builtin/grep.c +14 more+44 -881 reply
6/8branch: split off function that writes tracking info and commit subjectbuiltin/branch.c+27 -210 replies
7/8cherry: split off function to print output linesbuiltin/log.c+18 -160 replies
8/8diff --submodule: split into bite-sized piecessubmodule.c+61 -422 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/8] more warnings and cleanups

Here are some more patches in the same vein.

Patches 1-4 fix style problems noticed with make CC="c89 -pedantic" (mostly concerning the -pedantic rather than the c89).

Patch 5 updates fast-import.c (and everyone else) to put braces on the same line as "struct foo" --- the waste of space had bothered me every time I hacked on fast-import.c so now is my chance. :)

Patches 6-8 factor out some new functions to un-indent code a little. The indented code was discovered in changing

	struct pretty_print_ctx ctx = {0};

into something more verbose to appease gcc -Wmissing-field-initializers, which probably wasn't a good idea after all. :) But the by-product is nice.

Jonathan Nieder (8):
enums: omit trailing comma for portability
compat: make gcc bswap an inline function
svn-fe: do not use "return" for tail call returning void
vcs-svn: remove spurious semicolons
standardize brace placement in struct definitions
branch: split off function that writes tracking info and commit
subject
cherry: split off function to print output lines
diff --submodule: split into bite-sized pieces

 builtin/add.c         |    3 +-
 builtin/blame.c       |    3 +-
 builtin/branch.c      |   48 +++++++++++++----------
 builtin/grep.c        |    3 +-
 builtin/index-pack.c  |    6 +--
 builtin/log.c         |   34 +++++++++--------
 cache.h               |    2 +-
 commit.h              |    3 +-
 compat/bswap.h        |   18 +++++----
 config.c              |    3 +-
 convert.c             |    2 +-
 diff.c                |    6 +--
 fast-import.c         |   42 +++++++-------------
 http-push.c           |   15 ++-----
 http-walker.c         |    6 +--
 http.h                |   15 ++-----
 merge-recursive.c     |   12 ++----
 pack-check.c          |    3 +-
 string-list.h         |    3 +-
 submodule.c           |  103 +++++++++++++++++++++++++++++--------------------
 transport-helper.c    |    3 +-
 vcs-svn/repo_tree.c   |    2 +-
 vcs-svn/string_pool.c |    2 +-
 vcs-svn/svndump.c     |    3 +-
 24 files changed, 164 insertions(+), 176 deletions(-)