diff options
author | Arnold D. Robbins <arnold@skeeve.com> | 2013-08-22 21:52:11 +0300 |
---|---|---|
committer | Arnold D. Robbins <arnold@skeeve.com> | 2013-08-22 21:52:11 +0300 |
commit | 84f6bcd255d10b593298e0408011e9f03d1ab18c (patch) | |
tree | 7e462b90d7aa12586bb17571dddb88344a4e007a | |
parent | 844e16cea1ffe0e02b5b727fd8960271d1842020 (diff) | |
download | egawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.tar.gz egawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.tar.bz2 egawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.zip |
Clean up some -Wextra compile warnings in extensions.
-rw-r--r-- | extension/ChangeLog | 14 | ||||
-rw-r--r-- | extension/gawkfts.c | 2 | ||||
-rw-r--r-- | extension/inplace.c | 2 | ||||
-rw-r--r-- | extension/readdir.c | 3 | ||||
-rw-r--r-- | extension/revtwoway.c | 3 | ||||
-rw-r--r-- | extension/testext.c | 4 |
6 files changed, 27 insertions, 1 deletions
diff --git a/extension/ChangeLog b/extension/ChangeLog index d7e8e588..ff33673d 100644 --- a/extension/ChangeLog +++ b/extension/ChangeLog @@ -1,3 +1,17 @@ +2013-08-22 Arnold D. Robbins <arnold@skeeve.com> + + Clean up some warnings from -Wextra. + + * gawkfts.c (fts_set): Add cast to void for sp. + * inplace.c (at_exit): Add cast to void for data and exit_status. + * readdir.c (ftype): Add cast to void for dirname. + (dir_get_record): Assign NULL to *rt_start. + * revtwoway.c (rev2way_get_record): Add cast to void for errcode. + (rev2way_fwrite): Add cast to void for fp. + (rev2way_take_control_of): Add cast to void for name. + * testext.c (test_array_param, test_scalar, test_scalar_reserved, + test_indirect_vars): Add cast to void for nargs. + 2013-08-20 Arnold D. Robbins <arnold@skeeve.com> * gawkdirfd.h: Include ../nonposix.h to get FAKE_FD_VALUE. diff --git a/extension/gawkfts.c b/extension/gawkfts.c index 734dd7f7..81803b88 100644 --- a/extension/gawkfts.c +++ b/extension/gawkfts.c @@ -554,7 +554,7 @@ name: t = sp->fts_path + NAPPEND(p->fts_parent); int fts_set(FTS *sp, FTSENT *p, int instr) { - + (void) sp; /* silence warnings */ _DIAGASSERT(sp != NULL); _DIAGASSERT(p != NULL); diff --git a/extension/inplace.c b/extension/inplace.c index ad6f0e23..133b40f0 100644 --- a/extension/inplace.c +++ b/extension/inplace.c @@ -88,6 +88,8 @@ static struct { static void at_exit(void *data, int exit_status) { + (void) data; /* silence warnings */ + (void) exit_status; /* silence warnings */ if (state.tname) { unlink(state.tname); free(state.tname); diff --git a/extension/readdir.c b/extension/readdir.c index 061fe27f..91296801 100644 --- a/extension/readdir.c +++ b/extension/readdir.c @@ -89,6 +89,7 @@ static const char * ftype(struct dirent *entry, const char *dirname) { #ifdef DT_BLK + (void) dirname; /* silence warnings */ switch (entry->d_type) { case DT_BLK: return "b"; case DT_CHR: return "c"; @@ -158,6 +159,7 @@ get_inode(struct dirent *entry, const char *dirname) } return 0; #else + (void) dirname; /* silence warnings */ return entry->d_ino; #endif } @@ -215,6 +217,7 @@ dir_get_record(char **out, awk_input_buf_t *iobuf, int *errcode, *out = the_dir->buf; + *rt_start = NULL; *rt_len = 0; /* set RT to "" */ return len; } diff --git a/extension/revtwoway.c b/extension/revtwoway.c index 6e5bb710..5f490825 100644 --- a/extension/revtwoway.c +++ b/extension/revtwoway.c @@ -141,6 +141,7 @@ rev2way_get_record(char **out, awk_input_buf_t *iobuf, int *errcode, * error occurs. */ + (void) errcode; /* silence warnings */ if (out == NULL || iobuf == NULL || iobuf->opaque == NULL) return EOF; @@ -195,6 +196,7 @@ rev2way_fwrite(const void *buf, size_t size, size_t count, FILE *fp, void *opaqu size_t amount, char_count; char *src, *dest; + (void) fp; /* silence warnings */ if (opaque == NULL) return 0; /* error */ @@ -281,6 +283,7 @@ revtwoway_take_control_of(const char *name, awk_input_buf_t *inbuf, awk_output_b { two_way_proc_data_t *proc_data; + (void) name; /* silence warnings */ if (inbuf == NULL || outbuf == NULL) return awk_false; diff --git a/extension/testext.c b/extension/testext.c index df159575..d11272b8 100644 --- a/extension/testext.c +++ b/extension/testext.c @@ -537,6 +537,7 @@ test_array_param(int nargs, awk_value_t *result) awk_value_t new_array; awk_value_t arg0; + (void) nargs; /* silence warnings */ make_number(0.0, result); if (! get_argument(0, AWK_UNDEFINED, & arg0)) { @@ -615,6 +616,7 @@ test_scalar(int nargs, awk_value_t *result) awk_value_t new_value, new_value2; awk_value_t the_scalar; + (void) nargs; /* silence warnings */ make_number(0.0, result); if (! sym_lookup("the_scalar", AWK_SCALAR, & the_scalar)) { @@ -661,6 +663,7 @@ test_scalar_reserved(int nargs, awk_value_t *result) awk_value_t new_value; awk_value_t the_scalar; + (void) nargs; /* silence warnings */ make_number(0.0, result); /* look up a reserved variable - should pass */ @@ -712,6 +715,7 @@ test_indirect_vars(int nargs, awk_value_t *result) awk_value_t value; char *name = "NR"; + (void) nargs; /* silence warnings */ assert(result != NULL); make_number(0.0, result); |