more warnings and cleanups
Get series with b4
b4 am 20110316065256.GA5988@elieVersions newest first
- v1 Latest8 patches 8 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | enums: omit trailing comma for portabilitycache.h convert.c | +2 | 0 replies | |
| 2/8 | compat: make gcc bswap an inline functioncompat/ | +10 | 3 replies | |
| 3/8 | svn-fe: do not use "return" for tail call returning voidvcs-svn/ | +2 | 0 replies | |
| 4/8 | vcs-svn: remove spurious semicolonsvcs-svn/ vcs-svn/ | +2 | 2 replies | |
| 5/8 | standardize brace placement in struct definitionsbuiltin/ builtin/ builtin/ +14 more | +44 | 1 reply | |
| 6/8 | branch: split off function that writes tracking info and commit subjectbuiltin/ | +27 | 0 replies | |
| 7/8 | cherry: split off function to print output linesbuiltin/ | +18 | 0 replies | |
| 8/8 | diff --submodule: split into bite-sized piecessubmodule.c | +61 | 2 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
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(-)