From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Add void cast to StaticAssertExpr? |
Date: | 2018-02-15 18:04:52 |
Message-ID: | 20180215180452.xsb5lx5h32fjt4ed@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 2018-02-15 12:19:46 -0500, Tom Lane wrote:
> While poking around in buildfarm results, I noticed that some members are
> giving warnings like
>
> analyze.c:386:181: warning: expression result unused [-Wunused-value]
>
> which is apparently caused by the use of StaticAssertExpr in
>
> #define AllocSetContextCreate(parent, name, allocparams) \
> (StaticAssertExpr(__builtin_constant_p(name), \
> "Use AllocSetContextCreateExtended with MEMCONTEXT_COPY_NAME for non-constant context names"), \
> AllocSetContextCreateExtended(parent, name, 0, allocparams))
>
> Now, I could silence those warnings via
>
> - (StaticAssertExpr(__builtin_constant_p(name), \
> + ((void) StaticAssertExpr(__builtin_constant_p(name), \
>
> as I see has already been done in two similar uses of StaticAssertExpr in
> c.h. However, this seems a bit silly. Wouldn't it be better to put
> the void cast right into StaticAssertExpr?
No argument against it here.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2018-02-15 18:11:59 | Re: JIT compiling with LLVM v10.1 |
Previous Message | Andres Freund | 2018-02-15 18:01:57 | Re: spin.c includes pg_sema.h even if unnecessary |