Re: PATCH: zle -f from inside widget to set flags and make yank start/end zle params writable

[Mikael Magnusson <mikachu@xxxxxxxxx>]

On Fri, Oct 2, 2015 at 10:09 AM, Jun T. <takimoto-j@xxxxxxxxxxxxxxxxx> wrote:
> I've been using clang, but if I switch to gcc I get another warning:
>
> zle_thingy.c:671: warning: ‘zle_usable’ was used with no prototype before its definition
>
> Sorry for not reporting this with the previous one, but anyway
> this can be stopped by using func(void) instead of func():
>
> static int
> zle_usable(void)

I moved the definition up so it's not used before being defined instead.

> On 2015/10/02, at 4:43, Mikael Magnusson <mikachu@xxxxxxxxx> wrote:
>> I didn't realize configure
>> doesn't enable any warnings by default.
>
> configure adds -Wall -Wmissing-prototypes to CFLAGS only if CFLAGS
> is not specified by the user (lines 474 and below in configure.ac).
>
> I guess you specify CFLAGS by yourself (in environment?); then
> you need to explicitly include -Wxxx in CFLAGS.

I guess so, most projects add their warning flags regardless.

-- 
Mikael Magnusson