* Unnecessary #ifdef in commit c863114122ac
@ 2015-05-21 10:59 Andreas Ruprecht
2015-05-21 22:11 ` Jaegeuk Kim
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Ruprecht @ 2015-05-21 10:59 UTC (permalink / raw)
To: Jaegeuk Kim
Cc: Theodore Ts'o, Uday Savagaonkar, linux-kernel, Paul Bolle,
Valentin Rothberg, Stefan Hengelein
Hi Jaegeuk,
your commit c863114122ac ("f2fs crypto: add symlink encryption") showed
up in linux-next today (i.e., next-20150521). I noticed it because we
run a daily analysis on linux-next, checking for inconsistencies
regarding #ifdef blocks [0,1].
In the commit, you create nested #ifdef blocks like the following:
#ifdef CONFIG_F2FS_FS_ENCRYPTION
[...]
#ifdef CONFIG_F2FS_FS_XATTR
[...]
#endif
[...]
#endif
The inner #ifdef, however, is unnecessary as CONFIG_F2FS_FS_XATTR is
defined with a dependency on CONFIG_F2FS_FS_ENCRYPTION (line 78 in the
corresponding Kconfig file at fs/f2fs/Kconfig).
This means, CONFIG_F2FS_FS_XATTR can never be disabled if
CONFIG_F2FS_FS_ENCRYPTION is enabled, and that the inner #ifdef can
safely be removed.
Do you want me to send a patch for this or do you want to do this yourself?
Best regards,
Andreas
[0] https://cados.cs.fau.de
[1] http://www.linuxplumbersconf.org/2014/ocw/proposals/1863
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Unnecessary #ifdef in commit c863114122ac
2015-05-21 10:59 Unnecessary #ifdef in commit c863114122ac Andreas Ruprecht
@ 2015-05-21 22:11 ` Jaegeuk Kim
0 siblings, 0 replies; 2+ messages in thread
From: Jaegeuk Kim @ 2015-05-21 22:11 UTC (permalink / raw)
To: Andreas Ruprecht
Cc: Theodore Ts'o, Uday Savagaonkar, linux-kernel, Paul Bolle,
Valentin Rothberg, Stefan Hengelein
Hi Andreas,
On Thu, May 21, 2015 at 12:59:03PM +0200, Andreas Ruprecht wrote:
> Hi Jaegeuk,
>
> your commit c863114122ac ("f2fs crypto: add symlink encryption") showed
> up in linux-next today (i.e., next-20150521). I noticed it because we
> run a daily analysis on linux-next, checking for inconsistencies
> regarding #ifdef blocks [0,1].
>
> In the commit, you create nested #ifdef blocks like the following:
>
> #ifdef CONFIG_F2FS_FS_ENCRYPTION
> [...]
> #ifdef CONFIG_F2FS_FS_XATTR
> [...]
> #endif
> [...]
> #endif
>
> The inner #ifdef, however, is unnecessary as CONFIG_F2FS_FS_XATTR is
> defined with a dependency on CONFIG_F2FS_FS_ENCRYPTION (line 78 in the
> corresponding Kconfig file at fs/f2fs/Kconfig).
>
> This means, CONFIG_F2FS_FS_XATTR can never be disabled if
> CONFIG_F2FS_FS_ENCRYPTION is enabled, and that the inner #ifdef can
> safely be removed.
>
> Do you want me to send a patch for this or do you want to do this yourself?
Agreed.
Let me please fix this in the original patch.
Thanks,
>
> Best regards,
>
> Andreas
>
> [0] https://cados.cs.fau.de
> [1] http://www.linuxplumbersconf.org/2014/ocw/proposals/1863
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-05-21 22:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-21 10:59 Unnecessary #ifdef in commit c863114122ac Andreas Ruprecht
2015-05-21 22:11 ` Jaegeuk Kim
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.