hi,
OK. new patch:
o) old lines not touched
o) rename -W -> -Wextra
o) new line:
-Wextra -Wimplicit -Wreturn-type -Wunused -Wformat
Mike
On Mon, Feb 8, 2016 at 11:27 PM, Mike Pechkin
wrote:
> hi,
>
> you are correct, this is draft patch.
> > Goal: compile with minimum
oops
o) new line is:
-Wall -Wno-parentheses -Wno-sequence-point \
On Tue, Feb 9, 2016 at 10:50 PM, Mike Pechkin
wrote:
> hi,
>
> OK. new patch:
> o) old lines not touched
> o) rename -W -> -Wextra
> o) new line:
> -Wextra -Wimplicit -Wreturn-type -Wunused -Wformat
>
>
Hi Mike,
> Goal: compile with minimum commands + -Wall
> Patch attached.
> 4 new words, others not required.
> compiles without warnings and pass selftests on modern Linux.
Thanks! I've tried on Linux 32-bits, and it works and runs.
Fixing the warnings is OK, but why did you remove this?
hi,
you are correct, this is draft patch.
> Goal: compile with minimum commands + -Wall
as you can see patch supports goal, and pil32 pass selftests and my code.
Mike
hi all,
Goal: compile with minimum commands + -Wall
Patch attached.
4 new words, others not required.
compiles without warnings and pass selftests on modern Linux.
https://gcc.gnu.org/onlinedocs/gcc-4.7.4/gcc/Warning-Options.html#Warning-Options
1.
-Wall - says itself
2.
-Wno-parentheses for