On 04/03/14 19:47, Christian Couder wrote:
> This patch series implements a new command:
>
> git interpret-trailers
>
[snip]
Minor problem: this series causes sparse to complain, thus:
trailer.c:642:6: warning: symbol 'process_trailers' was not \
declared. Should it be static?
The following patch, on top of the 'pu' branch, fixes it:
--- >8 ---
Subject: [PATCH] trailer.c: suppress sparse warning
Check that the public interface, as declared in the trailer.h header
file, is consistent with the actual implementation. Add an #include
of the header file into the implementation file.
Noticed by sparse ("'process_trailers' was not declared. Should it
be static?").
Signed-off-by: Ramsay Jones <[email protected]>
---
trailer.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/trailer.c b/trailer.c
index b5de616..95d5874 100644
--- a/trailer.c
+++ b/trailer.c
@@ -1,6 +1,7 @@
#include "cache.h"
#include "run-command.h"
#include "argv-array.h"
+#include "trailer.h"
/*
* Copyright (c) 2013 Christian Couder <[email protected]>
*/
--
1.9.0
--- 8< ---
However, for this to work, in addition to squashing the above patch into
your patch #6, you would need to move the creation of the trailer.h header
file from patch 07/11 ("trailer: add interpret-trailers command") to patch
06/11 ("trailer: put all the processing together and print"), where it should
have been anyway! :-D
HTH
ATB,
Ramsay Jones
> Christian Couder (11):
> Add data structures and basic functions for commit trailers
> trailer: process trailers from stdin and arguments
> trailer: read and process config information
> trailer: process command line trailer arguments
> trailer: parse trailers from stdin
> trailer: put all the processing together and print
> trailer: add interpret-trailers command
> trailer: add tests for "git interpret-trailers"
> trailer: execute command from 'trailer.<name>.command'
> trailer: add tests for commands in config file
> Documentation: add documentation for 'git interpret-trailers'
>
> .gitignore | 1 +
> Documentation/git-interpret-trailers.txt | 123 ++++++
> Makefile | 2 +
> builtin.h | 1 +
> builtin/interpret-trailers.c | 33 ++
> git.c | 1 +
> t/t7513-interpret-trailers.sh | 261 ++++++++++++
> trailer.c | 661
> +++++++++++++++++++++++++++++++
> trailer.h | 6 +
> 9 files changed, 1089 insertions(+)
> create mode 100644 Documentation/git-interpret-trailers.txt
> create mode 100644 builtin/interpret-trailers.c
> create mode 100755 t/t7513-interpret-trailers.sh
> create mode 100644 trailer.c
> create mode 100644 trailer.h
>
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
