[PATCH 3/3] cli: fix clang build warnings
Tomi Ollila
tomi.ollila at iki.fi
Mon Feb 23 11:11:32 PST 2015
On Mon, Feb 23 2015, Jani Nikula <jani at nikula.org> wrote:
> Fix the following warning produced by clang 3.5.0:
>
> notmuch-search.c:730:25: warning: initializing 'void *' with an
> expression of type 'const notmuch_opt_desc_t (*)[4]' discards
> qualifiers [-Wincompatible-pointer-types-discards-qualifiers]
> { NOTMUCH_OPT_INHERIT, &common_options, NULL, 0, 0 },
> ^~~~~~~~~~~~~~~
> 1 warning generated.
>
> ---
>
> I'm not really happy with this one. An alternative would be to drop
> const from common_options, but I'm not happy with that either. And I'm
> not happy with the warning. :(
This looks tolerable to be and the whole series good. tests pass.
> ---
> notmuch-search.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/notmuch-search.c b/notmuch-search.c
> index 14b9f01c5ad1..a591d45b4c39 100644
> --- a/notmuch-search.c
> +++ b/notmuch-search.c
> @@ -671,7 +671,7 @@ notmuch_search_command (notmuch_config_t *config, int argc, char *argv[])
> { NOTMUCH_OPT_INT, &ctx->offset, "offset", 'O', 0 },
> { NOTMUCH_OPT_INT, &ctx->limit, "limit", 'L', 0 },
> { NOTMUCH_OPT_INT, &ctx->dupe, "duplicate", 'D', 0 },
> - { NOTMUCH_OPT_INHERIT, &common_options, NULL, 0, 0 },
> + { NOTMUCH_OPT_INHERIT, (void *) &common_options, NULL, 0, 0 },
> { 0, 0, 0, 0, 0 }
> };
>
> @@ -727,7 +727,7 @@ notmuch_address_command (notmuch_config_t *config, int argc, char *argv[])
> (notmuch_keyword_t []){ { "true", NOTMUCH_EXCLUDE_TRUE },
> { "false", NOTMUCH_EXCLUDE_FALSE },
> { 0, 0 } } },
> - { NOTMUCH_OPT_INHERIT, &common_options, NULL, 0, 0 },
> + { NOTMUCH_OPT_INHERIT, (void *) &common_options, NULL, 0, 0 },
> { 0, 0, 0, 0, 0 }
> };
>
> --
> 2.1.4
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list