On Mon, Jul 25, 2016 at 07:15:24PM +0900, Masahiro Yamada wrote:
The tools/moveconfig.py has a feature to cleanup #define/#undef's of moved config options, but I want this tool to do a better job.
For example, when we are moving CONFIG_FOO and its define is surrounded by #ifdef ... #endif, like follows:
#ifdef CONFIG_BAR # define CONFIG_FOO #endif
The header cleanup will leave empty #ifdef ... #endif:
#ifdef CONFIG_BAR #endif
Likewise, if a define line between two blank lines
<blank line> #define CONFIG_FOO <blank lines.
... is deleted, the result of the clean-up will be successive empty lines, which is a coding-style violation.
It is tedious to remove left-over garbage lines manually, so I want the tool to take care of this. The tool's job is still not perfect, so we should check the output of the tool, but I hope our life will be much easier with this patch.
Signed-off-by: Masahiro Yamada yamada.masahiro@socionext.com
Applied to u-boot/master, thanks!