Date: Wed, 28 Mar 2001 02:34:50 -0500 (EST) From: Robert Watson <rwatson@FreeBSD.org> To: Warner Losh <imp@harmony.village.org> Cc: Jeroen Ruigrok/Asmodai <asmodai@wxs.nl>, "Andrey A. Chernov" <ache@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/tail forward.c Message-ID: <Pine.NEB.3.96L.1010328023423.96088J-100000@fledge.watson.org> In-Reply-To: <200103280732.f2S7We952543@harmony.village.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 28 Mar 2001, Warner Losh wrote: > In message <20010328064206.A90260@daemon.ninth-circle.org> Jeroen Ruigrok/Asmodai writes: > : -On [20010328 01:30], Andrey A. Chernov (ache@FreeBSD.org) wrote: > : > fseeko(file_size, SEEK_SET) -> fseek(0L, SEEK_END) > : > : Shouldn't 0L actually be NULL? > > No. fseek takes a long offset. fseek(fp, 0L, SEEK_END) is correct. So clearly the reality of the patch differed from the comment, because your code snippet != comment code snippet. Robert N M Watson FreeBSD Core Team, TrustedBSD Project robert@fledge.watson.org NAI Labs, Safeport Network Services To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.NEB.3.96L.1010328023423.96088J-100000>