Subject: Re: CVS commit: src/usr.bin/vi
To: Luke Mewburn <lukem@netbsd.org>
From: Thomas Klausner <wiz@NetBSD.org>
List: source-changes
Date: 06/02/2005 10:55:34
On Thu, Jun 02, 2005 at 04:25:16AM +0000, Luke Mewburn wrote:
> 
> Module Name:	src
> Committed By:	lukem
> Date:		Thu Jun  2 04:25:16 UTC 2005
> 
> Modified Files:
> 	src/usr.bin/vi/common: cut.c options.c recover.c
> 	src/usr.bin/vi/ex: ex_cscope.c ex_join.c ex_read.c ex_script.c ex_tag.c
> 	    ex_write.c
> 	src/usr.bin/vi/vi: v_txt.c vs_line.c vs_refresh.c vs_relative.c
> 	    vs_split.c
> 
> Log Message:
> appease gcc -Wuninitialized.
> Each is marked with XXXGCC, because in some cases it looks like gcc
> isn't correctly detecting that
> 	for ( ; xxx ; yyy)
> 		foo = 0;
> always sets foo ...

That depends on xxx. On 3.99.5/i386, running
	#include <stdio.h>
	main() { int y=1, x=5; for (;x<5;x++) y=0; printf("%d\n", y); }
prints "1", not 0.
 Thomas