Re: [BBv2] warning support

From: Alexey Pakhunov (alexeypa_at_[hidden])
Date: 2005-09-17 11:52:18

Reece Dunn wrote:

> +feature warnings :
> + default # use the compilers default warning settings
> + on # turn warnings on
> + off # turn warnings off
> + all # enable all warnings
> + : propagated ;
> +

In my understanding the levels should be off -> default -> all. 
Additional intermediate levels may be defined but 'on' and 'default' are 
synonyms.

Also the feature should not be 'propagated'. It should be 'free' because 
it does not affect the result of compilation. It only affects the 
warnings that will be displayed.

> +feature warnings-as-errors : off on : propagated ;

This one should be 'free' as well.

> +flags borland.compile OPTIONS <warnings>on : -w ;
> +flags borland.compile OPTIONS <warnings>all : -w ;
> +flags borland.compile OPTIONS <warnings-as-errors>on : -w! ;

Can 'OPTIONS' include both '-w' and '-w!' at the same time? We may have 
both <warnings>on and <warnings-as-errors>on simultaneously.

> +flags cw.compile CFLAGS <warnings>on : -w on ;
> +flags cw.compile CFLAGS <warnings>off : -w off ;
> +flags cw.compile CFLAGS <warnings>all : -w all ;
> +flags cw.compile CFLAGS <warnings-as-errors>on : -w error ;

The same thing: are all four options mutually exclusive?

Best regards/Venlig hilsen,
Alexey Pakhunov.