Re: [PATCH] CONFIG_* In Comments Considered Harmful

From: Jörn Engel
Date: Wed Oct 01 2003 - 09:44:28 EST


On Wed, 1 October 2003 10:29:55 -0400, Valdis.Kletnieks@xxxxxx wrote:
> On Wed, 01 Oct 2003 15:19:52 BST, Dave Jones said:
>
> > Maybe it should be taught to parse comments? There are zillions of
> > #endif /* CONFIG_FOO */
> > braces in the tree. Why is this one special ?
>
> I think it's because it looked like:
>
> #ifdef CONFIG_FOO
> ....
> #endif /* CONFIG_FOO or CONFIG_BAR */
>
> and it concluded there was a dependency on BAR.

Or rather like this:

#ifdef CONFIG_FOO
...
#endif /* CONFIG_FO */

Problem is that we humans correct the type before it even reaches our
conciousness. Machines don't do that yet.

Jörn

--
The strong give up and move away, while the weak give up and stay.
-- unknown
-
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/