quote arguments in macros?

[Matt Caswell]

Yes - they should. Fixup PR welcome.

Matt

On 07/04/2020 12:12, Claus Assmann wrote:
> I just got a compiler warning while modifying some code:
> 
> SSL_set_tlsext_host_name(ssl, sni)
> ->
> 
> #define IS_EMPTY(s)  (NULL == (s) || '\0' == *(s))
> SSL_set_tlsext_host_name(ssl, !IS_EMPTY(sni) ? sni : other)
> 
> warning: cast to 'char *' from smaller integer type 'int'
> 
>       'SSL_set_tlsext_host_name'
> SSL_ctrl(s,SSL_CTRL_SET_TLSEXT_HOSTNAME,TLSEXT_NAMETYPE_host_name,(char *)name)
>                                                                           ^^^^
> 
> Shouldn't the arguments in those macros be quoted, e.g.,
> SSL_ctrl((s),SSL_CTRL_SET_TLSEXT_HOSTNAME,TLSEXT_NAMETYPE_host_name,(char *)(name))
> ?
>