aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnold D. Robbins <arnold@skeeve.com>2013-08-22 21:52:11 +0300
committerArnold D. Robbins <arnold@skeeve.com>2013-08-22 21:52:11 +0300
commit84f6bcd255d10b593298e0408011e9f03d1ab18c (patch)
tree7e462b90d7aa12586bb17571dddb88344a4e007a
parent844e16cea1ffe0e02b5b727fd8960271d1842020 (diff)
downloadegawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.tar.gz
egawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.tar.bz2
egawk-84f6bcd255d10b593298e0408011e9f03d1ab18c.zip
Clean up some -Wextra compile warnings in extensions.
-rw-r--r--extension/ChangeLog14
-rw-r--r--extension/gawkfts.c2
-rw-r--r--extension/inplace.c2
-rw-r--r--extension/readdir.c3
-rw-r--r--extension/revtwoway.c3
-rw-r--r--extension/testext.c4
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);