#9917: ECL has too few arguments and two many on file dpp.c
------------------------+---------------------------------------------------
Reporter: drkirkby | Owner: GeorgSWeber
Type: defect | Status: new
Priority: major | Milestone: sage-4.6
Component: build | Keywords: ecl
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------+---------------------------------------------------
When I'm building ecl-10.2.1 as part of Sage I get too warning messages
from gcc.
/export/home/drkirkby/sage-4.6.alpha0/spkg/build/ecl-10.2.1.p2/src/src/c/dpp.c:
In function 'put_declaration':
/export/home/drkirkby/sage-4.6.alpha0/spkg/build/ecl-10.2.1.p2/src/src/c/dpp.c:678:5:
warning: too few arguments for format
/export/home/drkirkby/sage-4.6.alpha0/spkg/build/ecl-10.2.1.p2/src/src/c/dpp.c:680:13:
warning: too many arguments for format
Looking at line 678 of dpp.c, I see:
fprintf(out, "\tif (ecl_unlikely(narg!=%d))");
So there's a %d, but what is associate with the %d? There should be an
integer, but there is not one. So it seems to me gcc is right to complain
there are too few arguments for format.
Likewise, on line 680, I see:
fprintf(out, "\t FEwrong_num_arguments(MAKE_FIXNUM(%d));\n",
nreq, function_code);
There's one two arguments supplied, but only one %d is there. That does
not make any sense to me. Both "nreq" and "function_code" are declared as
integers, so should there not two %d's and not one?
Again, it seems gcc is right to complain that.
There are thousands of warning messages in Sage, but I'm a bit concerned
about resolving those in ecl, as the ecl library being built has text
relocation problems - see #9840
Dave
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9917>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
