From owner-cvs-all Mon Jul 8 8:48:59 2002 Delivered-To: cvs-all@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 931) id 577EF37B40C; Mon, 8 Jul 2002 08:48:48 -0700 (PDT) Date: Mon, 8 Jul 2002 08:48:48 -0700 From: Juli Mallett To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/bin/ps print.c Message-ID: <20020708084847.C39552@FreeBSD.ORG> References: <200207072146.g67Lk0nT041098@freefall.freebsd.org> <20020708121714.GF75407@starjuice.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20020708121714.GF75407@starjuice.net>; from sheldonh@starjuice.net on Mon, Jul 08, 2002 at 02:17:14PM +0200 Organisation: The FreeBSD Project X-Alternate-Addresses: , , , X-Affiliated-Projects: FreeBSD, xMach, ircd-hybrid-7 X-Towel: Yes X-LiveJournal: flata X-Negacore: Yes X-Warning: If you make me read stupid email, don't be surprised if I respond. If you don't want me to reply to something, don't send it to me. Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG * De: Sheldon Hearn [ Data: 2002-07-08 ] [ Subjecte: Re: cvs commit: src/bin/ps print.c ] > On (2002/07/07 14:45), Juli Mallett wrote: > > > Modified files: > > bin/ps print.c > > Log: > > Make printval() take a 'void *' thus negating any assumptions the compiler > > may try to make about the alignment of the dereferenced datum. > > Doesn't this break buildworld with warnings promoted to errors? Lack of downgrading of -Wformat does that, and this code relies on that breakage, and my changes had no effect on the code in question, in the manner in question. The commit that broke this was when I removed WFORMAT explicit setting because 'make WARNS=6' never gave me trouble, and such, and I forgot that WARNS doesnt imply WFORMAT. Oops. I'll back that part of it out. Thanks. > ===> bin/ps > [...] > cc -O -pipe -march=pentiumpro -DLAZY_PS -Werror -Wall -Wno-format-y2k -Wno-uninitialized -Wformat=2 -Wno-format-extra-args -Werror -c /usr/src/bin/ps/print.c > cc1: warnings being treated as errors > /usr/src/bin/ps/print.c: In function `started': > /usr/src/bin/ps/print.c:400: warning: `%y' yields only last 2 digits of year > /usr/src/bin/ps/print.c: In function `lstarted': > /usr/src/bin/ps/print.c:417: warning: `%c' yields only last 2 digits of year in some locales on non-BSD systems > /usr/src/bin/ps/print.c: In function `printval': > /usr/src/bin/ps/print.c:686: warning: format not a string literal, argument types not checked > /usr/src/bin/ps/print.c:689: warning: format not a string literal, argument types not checked > [...] > > Ciao, > Sheldon. -- Juli Mallett | FreeBSD: The Power To Serve Will break world for fulltime employment. | finger jmallett@FreeBSD.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message