Re: [PATCH 5/6] net: tftp: use IS_ENABLED(CONFIG_NET_TFTP_VARS) instead of #if

2022-11-28 Thread Tom Rini
On Fri, Oct 14, 2022 at 07:43:41PM +0200, Rasmus Villemoes wrote: > Nothing inside this block depends on NET_TFTP_VARS to be set to parse > correctly. Switch to C if() in preparation for adding code before > this (to avoid a declaration-after-statement warning). > > Signed-off-by: Rasmus

Re: [PATCH 5/6] net: tftp: use IS_ENABLED(CONFIG_NET_TFTP_VARS) instead of #if

2022-10-17 Thread Rasmus Villemoes
On 16/10/2022 20.28, Ramon Fried wrote: > On Fri, Oct 14, 2022 at 8:44 PM Rasmus Villemoes > wrote: >> >> Nothing inside this block depends on NET_TFTP_VARS to be set to parse >> correctly. Switch to C if() in preparation for adding code before >> this (to avoid a declaration-after-statement

Re: [PATCH 5/6] net: tftp: use IS_ENABLED(CONFIG_NET_TFTP_VARS) instead of #if

2022-10-16 Thread Ramon Fried
On Fri, Oct 14, 2022 at 8:44 PM Rasmus Villemoes wrote: > > Nothing inside this block depends on NET_TFTP_VARS to be set to parse > correctly. Switch to C if() in preparation for adding code before > this (to avoid a declaration-after-statement warning). What's the motivation here ? The #ifdef is

[PATCH 5/6] net: tftp: use IS_ENABLED(CONFIG_NET_TFTP_VARS) instead of #if

2022-10-14 Thread Rasmus Villemoes
Nothing inside this block depends on NET_TFTP_VARS to be set to parse correctly. Switch to C if() in preparation for adding code before this (to avoid a declaration-after-statement warning). Signed-off-by: Rasmus Villemoes --- net/tftp.c | 56