From owner-freebsd-current Mon Aug 24 11:54:00 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id LAA16148 for freebsd-current-outgoing; Mon, 24 Aug 1998 11:54:00 -0700 (PDT) (envelope-from owner-freebsd-current@FreeBSD.ORG) Received: from tim.xenologics.com (tim.xenologics.com [194.77.5.24]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id LAA16119 for ; Mon, 24 Aug 1998 11:53:50 -0700 (PDT) (envelope-from seggers@semyam.dinoco.de) Received: (from uucp@localhost) by tim.xenologics.com (8.8.5/8.8.8) with UUCP id UAA09041; Mon, 24 Aug 1998 20:51:34 +0200 (MET DST) Received: from semyam.dinoco.de (semyam.dinoco.de [127.0.0.1]) by semyam.dinoco.de (8.9.1/8.8.8) with ESMTP id UAA25824; Mon, 24 Aug 1998 20:40:10 +0200 (CEST) (envelope-from seggers@semyam.dinoco.de) Message-Id: <199808241840.UAA25824@semyam.dinoco.de> To: Garrett Wollman cc: Kris Kennaway , Mike Smith , freebsd-current@FreeBSD.ORG, seggers@semyam.dinoco.de Subject: Re: lpr code cleanup? (was: Re: gcc 2.8 ) In-reply-to: Your message of "Sun, 23 Aug 1998 13:40:06 EDT." <199808231740.NAA23791@khavrinen.lcs.mit.edu> Date: Mon, 24 Aug 1998 20:40:09 +0200 From: Stefan Eggers Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > The intent of this was to make sure that anyone who introduced > warnings after I had laboriously eliminated them would get pounded on > by the -current regulars for breaking the build. I took a look at the code and CWARNFLAGS didn't get used in all of the makefiles so I fixed that in a local copy of the lpr source - and of course corrected the complaints egcs (which I used for the test as it seems to be a little bit more picky than the system's gcc) gave me after doing that. In the course of doing this I found a misdeclared function. Nothing serious as it was the result type and the result wasn't used anyway. The "register varname;" definitions in some of the functions I changed to "register int varname;" which silenced egcs and is OK for K&R, too. Compiling now works for me with a recent egcs (a July snapshot) for the lpr subsystem. Don't know about the rest as I buildworld with gcc. I will now take a close look at my changes in the next few days, inte- grate it in my local source tree and then install the new executables. Maybe it still works afterward. ;-) Anybody interested in the changes I made can of course get a diff via email before I dare to send a PR. Stefan. -- Stefan Eggers Lu4 yao2 zhi1 ma3 li4, Max-Slevogt-Str. 1 ri4 jiu3 jian4 ren2 xin1. 51109 Koeln Federal Republic of Germany To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message