Mail Archives: geda-user/2014/09/23/23:05:53
Girvin Herr wrote:
> If there are warnings, what
> other, worse bugs are in there?
Lets see...
<set-gcc-flags-to -W -Wall -ansi -pedantic>
<build guile>
<grep log> (6k lines(!)) there are actually quite a few warnings:
(...)
read.c: In function 'try_read_ci_chars':
read.c:983:3: warning: implicit declaration of function 'alloca' [-Wimplicit-function-declaration]
char *chars_read = alloca (num_chars_wanted);
^
read.c:983:22: warning: incompatible implicit declaration of built-in function 'alloca'
char *chars_read = alloca (num_chars_wanted);
^
posix.c: In function 'scm_execl':
posix.c:1144:3: warning: passing argument 2 of 'execv' from incompatible pointer type
execv (exec_file, exec_argv);
^
In file included from /usr/local/src/mxe/usr/i686-pc-mingw32.static/include/unistd.h:13:0,
from ../lib/unistd.h:40,
from posix.c:50:
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/process.h:118:42: note: expected 'const char * const*' but argument is of type 'char **'
_CRTIMP intptr_t __cdecl __MINGW_NOTHROW execv (const char*, const char* const*);
^
posix.c: In function 'scm_execlp':
posix.c:1173:3: warning: passing argument 2 of 'execvp' from incompatible pointer type
execvp (exec_file, exec_argv);
^
In file included from /usr/local/src/mxe/usr/i686-pc-mingw32.static/include/unistd.h:13:0,
from ../lib/unistd.h:40,
from posix.c:50:
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/process.h:120:42: note: expected 'const char * const*' but argument is of type 'char **'
_CRTIMP intptr_t __cdecl __MINGW_NOTHROW execvp (const char*, const char* const*);
^
posix.c: In function 'scm_execle':
posix.c:1207:3: warning: passing argument 2 of 'execve' from incompatible pointer type
execve (exec_file, exec_argv, exec_env);
^
In file included from /usr/local/src/mxe/usr/i686-pc-mingw32.static/include/unistd.h:13:0,
from ../lib/unistd.h:40,
from posix.c:50:
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/process.h:119:42: note: expected 'const char * const*' but argument is of type 'char **'
_CRTIMP intptr_t __cdecl __MINGW_NOTHROW execve (const char*, const char* const*, const char* const*);
^
posix.c:1207:3: warning: passing argument 3 of 'execve' from incompatible pointer type
execve (exec_file, exec_argv, exec_env);
^
In file included from /usr/local/src/mxe/usr/i686-pc-mingw32.static/include/unistd.h:13:0,
from ../lib/unistd.h:40,
from posix.c:50:
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/process.h:119:42: note: expected 'const char * const*' but argument is of type 'char **'
_CRTIMP intptr_t __cdecl __MINGW_NOTHROW execve (const char*, const char* const*, const char* const*);
(...)
The next warnings look more like scheme issues, rather than c:
ice-9/session.scm:241:11: warning: non-literal format string
(...)
language/ecmascript/function.scm:40:9: warning: possibly unbound variable `<js-array-object>'
language/ecmascript/function.scm:44:43: warning: possibly unbound variable `js-array-vector'
(...)
language/ecmascript/base.scm:227:22: warning: possibly unbound variable `Boolean'
language/ecmascript/base.scm:228:21: warning: possibly unbound variable `String'
language/ecmascript/base.scm:229:21: warning: possibly unbound variable `Number'
(...)
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:63:0: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:97:49: warning: possibly unbound variable `%readline'
ice-9/readline.scm:125:38: warning: possibly unbound variable `readline-options-interface'
ice-9/readline.scm:156:4: warning: possibly unbound variable `%readline'
ice-9/readline.scm:206:4: warning: possibly unbound variable `*readline-completion-function*'
ice-9/readline.scm:210:2: warning: possibly unbound variable `*readline-completion-function*'
ice-9/readline.scm:213:10: warning: possibly unbound variable `*readline-completion-function*'
ice-9/readline.scm:216:10: warning: possibly unbound variable `*readline-completion-function*'
(...)
and finally the long long warnings I mentioned before:
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/guile/2.0/libguile/deprecated.h:690:44: warning: ISO C90 does not support 'long long' [-Wlong-long]
SCM_DEPRECATED SCM scm_long_long2num (long long sl);
^
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/guile/2.0/libguile/deprecated.h:691:54: warning: ISO C90 does not support 'long long' [-Wlong-long]
SCM_DEPRECATED SCM scm_ulong_long2num (unsigned long long sl);
^
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/guile/2.0/libguile/deprecated.h:692:21: warning: ISO C90 does not support 'long long' [-Wlong-long]
SCM_DEPRECATED long long scm_num2long_long (SCM num, unsigned long int pos,
^
/usr/local/src/mxe/usr/i686-pc-mingw32.static/include/guile/2.0/libguile/deprecated.h:694:30: warning: ISO C90 does not support 'long long' [-Wlong-long]
SCM_DEPRECATED unsigned long long scm_num2ulong_long (SCM num, unsigned long int pos,
\--------------------
Ok, this seems less overwhelming than I thought. The initial c warnings look
familiar. From my scarce coding experiences long long ago "incompatible
pointer type" and "incompatible implicit declarations" ring a serious bell.
I'll accept the challenge and try to (re)learn as I go.
Don't know what to think about the scheme warnings.
Will keep you updated. Just don't hold your breath. I suck at coding and there
is a day job, too...
---<)kaimartin(>---
- Raw text -