On Wed, Feb 22, 2017 at 11:30 PM, Luc Van Oostenryck
<luc.vanoostenryck@xxxxxxxxx> wrote:
> When an array is initialized, it may be convenient to first
> initialize all entries with some default value via the
> '[a ... b]' notation and then override some of these entries
> with a non-default value. Unfortunately, this, of course,
> is not compatible with the default warning flag '-Woverride-init'.
Looks good to me.
> + if (!a && !Woverride_init_whole_range) {
> + // If first entry is the whole range, do not let
> + // any warning about it (this allow to initialize
> + // an array with some default value and then override
> + // some specific entries).
You might want to use some C style multi line comment /* ... */
Using // for multiple line is a bit strange.
Applied to sparse-next, but if you have update version I will
apply that as well.
Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
