aboutsummaryrefslogtreecommitdiff
path: root/latin.c
diff options
context:
space:
mode:
authorBen Harris <bjh21@bjh21.me.uk>2023-02-16 21:21:15 +0000
committerBen Harris <bjh21@bjh21.me.uk>2023-02-18 00:13:15 +0000
commit0186d78da9e83103eb80b3814d4de8031f29232f (patch)
tree26774b7be7540d980a1ee5cba5d83445a528bd5b /latin.c
parenta7e738aceb36e4a70d41cf09a74b2d7a3af6cbe0 (diff)
downloadpuzzles-0186d78da9e83103eb80b3814d4de8031f29232f.zip
puzzles-0186d78da9e83103eb80b3814d4de8031f29232f.tar.gz
puzzles-0186d78da9e83103eb80b3814d4de8031f29232f.tar.bz2
puzzles-0186d78da9e83103eb80b3814d4de8031f29232f.tar.xz
Mark many more function (and some objects) static
I noticed commit db3b531e2cab765a00475054d2e9046c9d0437d3 in the history where Simon added a bunch of "static" qualifiers. That suggested that consistently marking internal functions "static" is desirable, so I tried a build using GCC's -Wmissing-declarations, which requires prior declaration (presumed to be in a header file) of all global functions. This commit makes the GTK build clean under GCC's -Wmissing-declarations. I've also adding "static" to a few obviously internal objects, but GCC doesn't complain about those so I certainly haven't got them all.
Diffstat (limited to 'latin.c')
-rw-r--r--latin.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/latin.c b/latin.c
index 59f306d..2b4848f 100644
--- a/latin.c
+++ b/latin.c
@@ -1352,7 +1352,7 @@ static void gen(int order, random_state *rs, int debug)
sfree(sq);
}
-void test_soak(int order, random_state *rs)
+static void test_soak(int order, random_state *rs)
{
digit *sq;
int n = 0;
@@ -1375,7 +1375,7 @@ void test_soak(int order, random_state *rs)
}
}
-void usage_exit(const char *msg)
+static void usage_exit(const char *msg)
{
if (msg)
fprintf(stderr, "%s: %s\n", quis, msg);