Re: [PATCH 4/4] kconfig: remove unused definition from scanner

From: Yann E. MORIN
Date: Thu Oct 03 2013 - 02:38:10 EST


Martin, All,

On Thursday 03 October 2013 02:30:21 Martin Walch wrote:
> On Wednesday 02 October 2013 08:57:54 Yann E. MORIN wrote:
> > On Wednesday 02 October 2013 08:42:59 Dirk Gouders wrote:
> > > I think you should also regenerate the scanner and add
> > > the new zconf.lex.c_shipped to this patch.
> >
> > I think sending the re-generated parser as a separate patch helps
> > in reviewing. Ie. I'd prefer a fifth patch.
> >
> > BTW, regenerating the parser can be done with:
> > make REGENERATE_PARSERS=1 config
>
> Actually, I regenerated the scanner. However, the results did not change
> after applying the patch, because the definition that the patch removes
> (ws) is not used anywhere. It is only a cleanup.

OK, then please say so in the commit message.

Regards,
Yann E. MORIN.

--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +0/33 662376056 | Software Designer | \ / CAMPAIGN | ^ |
| --==< O_o >==-- '------------.-------: X AGAINST | /e\ There is no |
| http://ymorin.is-a-geek.org/ | (*_*) | / \ HTML MAIL | """ conspiracy. |
'------------------------------'-------'------------------'--------------------'
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/