lkml.org 
[lkml]   [2004]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: Trivial Comment Patch: 2.6.6-mm3
From
Date
Chris,

On Tue, 2004-05-18 at 16:07, Chris Wedgwood wrote:
> On Tue, May 18, 2004 at 09:08:57AM -0400, Danny Cox wrote:
>
> > This fixes a comment in the stack overflow check that's wrong with
> > 4K stacks.
>
> > #ifdef CONFIG_DEBUG_STACKOVERFLOW
> > - /* Debugging check for stack overflow: is there less than 1KB free? */
> > + /* Debugging check for stack overflow: is there less than 512B free? */
> > {
> > long esp;
>
> akpm, how about just yank the line? the comment's sanity depends on a
> CONFIG option and also a value in another file from a far off land.

IMHO, I'd say keep the line, but remove the offensive part (everything
past the ':', inclusive).

--
Daniel S. Cox
Electronic Commerce Systems

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:03    [W:0.043 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site