From b4cd692bae02a62816f7ed11d1bc9d98bd9a4281 Mon Sep 17 00:00:00 2001 From: Landon Curt Noll Date: Wed, 21 Feb 2018 12:37:46 -0800 Subject: [PATCH] Fixed compiler warnings Fixed some compiler warnings. Added work around for a gcc warning bug. --- CHANGES | 15 +++++++++++++-- calc.c | 10 ++++++---- value.c | 2 +- 3 files changed, 20 insertions(+), 7 deletions(-) diff --git a/CHANGES b/CHANGES index 5ddeb92..56c468d 100644 --- a/CHANGES +++ b/CHANGES @@ -1,7 +1,18 @@ -The following are the changes from calc version 2.12.6.5 to date: +The following are the changes from calc version 2.12.6.6 to date: + + For historical purposes, in lucas.cal, gen_v1(1, n) always returns 4. + + Fixed some compiler warnings, thanks to a report by Mike + . + + Added work around for a gcc warning bug, thanks to a report by Mike + . + + +The following are the changes from calc version 2.12.6.4 to 2.12.6.5: Fixed warning about undefined operations involving the qlink(q) - macro by deplacing that macro with an inline-function. Thanks goes + macro by replacing that macro with an inline-function. Thanks goes to David Haller for this fix. NOTE for Windows 10 users: Pavel Nemec diff --git a/calc.c b/calc.c index ecd098a..cb7dfb8 100644 --- a/calc.c +++ b/calc.c @@ -104,6 +104,7 @@ main(int argc, char **argv) int c; /* option */ int index; int maxindex; + int unusedint = 0; /* avoids gcc compiler warning bug - '...assigned, but never used...' */ char *cp; char *endcp; char *bp; @@ -278,7 +279,7 @@ main(int argc, char **argv) exit(6); } calc_debug = cp; - (void) strtol(cp, &endcp, 10); + unusedint = strtol(cp, &endcp, 10); /* avoids gcc compiler warning bug */ cp = endcp; if (*cp != '\0' && *cp != ' ' && *cp != ':') { @@ -310,7 +311,7 @@ main(int argc, char **argv) exit(9); } resource_debug = cp; - (void) strtol(cp, &endcp, 10); + unusedint = strtol(cp, &endcp, 10); /* avoids gcc compiler warning bug */ cp = endcp; if (*cp != '\0' && *cp != ' ' && *cp != ':') { @@ -340,7 +341,7 @@ main(int argc, char **argv) exit(12); } user_debug = cp; - (void) strtol(cp, &endcp, 10); + unusedint = strtol(cp, &endcp, 10); /* avoids gcc compiler warning bug */ cp = endcp; if (*cp != '\0' && *cp != ' ') { fprintf(stderr, "Bad syntax in" @@ -721,8 +722,9 @@ main(int argc, char **argv) printf("main: run_state = %s\n", run_state_name(run_state)); /* - * all done + * All done! - Jessica Noll, Age 2 */ + unusedint++; /* avoids gcc compiler warning bug */ libcalc_call_me_last(); return (run_state == RUN_EXIT_WITH_ERROR || run_state == RUN_ZERO) ? 1 : 0; diff --git a/value.c b/value.c index bc47d25..95c0917 100644 --- a/value.c +++ b/value.c @@ -2944,7 +2944,7 @@ printestr(VALUE *vp) bp = vp->v_nblock->blk; } i = bp->datalen; - math_fmt("%ld,%d)", i, bp->blkchunk); + math_fmt("%ld,%d)", i, (int) bp->blkchunk); cp = bp->data; if (i > 0) { math_str("={");