diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2010-03-09 10:49:33 +1100 |
---|---|---|
committer | Jon Loeliger <jdl@jdl.com> | 2010-03-10 08:12:28 -0600 |
commit | 74f6a51b1f71ffc7d9557e063630469a93eeddec (patch) | |
tree | c9f4bc016350af9bbd9b2b671d8aa4f687e7f9fc /tests | |
parent | 8765874963852b2733777e69686251205238ad3d (diff) | |
download | device-tree-compiler-74f6a51b1f71ffc7d9557e063630469a93eeddec.tar.gz |
dtc: Enable more warnings
This patch turns on a bunch of extra gcc warnings, most of which are
probably a good idea. Of the new warnings -Wnested-externs and
-Wstrict-prototypes need no code changes, we're already warning-clean.
The remaining one, -Wmissing-prototypes requires trivial changes in
some of the tests (making functions local).
This patch also rearranges the warnings flags into a separate make
variable for convenience, and turns on -Werror, to really encourage
people to keep the code warning-clean.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/extra-terminating-null.c | 2 | ||||
-rw-r--r-- | tests/get_alias.c | 2 | ||||
-rw-r--r-- | tests/path_offset_aliases.c | 2 | ||||
-rw-r--r-- | tests/value-labels.c | 4 |
4 files changed, 5 insertions, 5 deletions
diff --git a/tests/extra-terminating-null.c b/tests/extra-terminating-null.c index bb71b1a..8a2043f 100644 --- a/tests/extra-terminating-null.c +++ b/tests/extra-terminating-null.c @@ -28,7 +28,7 @@ #include "tests.h" #include "testdata.h" -void check_extranull(void *fdt, const char *prop, const char *str, int numnulls) +static void check_extranull(void *fdt, const char *prop, const char *str, int numnulls) { int len = strlen(str); char checkbuf[len+numnulls]; diff --git a/tests/get_alias.c b/tests/get_alias.c index 8eeaee4..1e0faf4 100644 --- a/tests/get_alias.c +++ b/tests/get_alias.c @@ -29,7 +29,7 @@ #include "tests.h" #include "testdata.h" -void check_alias(void *fdt, const char *path, const char *alias) +static void check_alias(void *fdt, const char *path, const char *alias) { const char *aliaspath; diff --git a/tests/path_offset_aliases.c b/tests/path_offset_aliases.c index 191edd2..3682da4 100644 --- a/tests/path_offset_aliases.c +++ b/tests/path_offset_aliases.c @@ -29,7 +29,7 @@ #include "tests.h" #include "testdata.h" -void check_alias(void *fdt, const char *full_path, const char *alias_path) +static void check_alias(void *fdt, const char *full_path, const char *alias_path) { int offset, offset_a; diff --git a/tests/value-labels.c b/tests/value-labels.c index c5aea8f..abe2721 100644 --- a/tests/value-labels.c +++ b/tests/value-labels.c @@ -59,8 +59,8 @@ struct val_label labels3[] = { { "end3", -1 }, }; -void check_prop_labels(void *sohandle, void *fdt, const char *name, - const struct val_label* labels, int n) +static void check_prop_labels(void *sohandle, void *fdt, const char *name, + const struct val_label* labels, int n) { const struct fdt_property *prop; const char *p; |