Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 30 Jan 2003 19:03:13 -0500
From:      Garance A Drosihn <drosih@rpi.edu>
To:        Tim Robbins <tjr@FreeBSD.org>, "M. Warner Losh" <imp@bsdimp.com>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/lib/libc/stdio gets.c
Message-ID:  <p05200f3bba5f6bea34cf@[128.113.24.47]>
In-Reply-To: <20030131092613.A48425@dilbert.robbins.dropbear.id.au>
References:  <200301301200.h0UC0Qfd081080@repoman.freebsd.org> <20030130.063524.24235010.imp@bsdimp.com> <20030131092613.A48425@dilbert.robbins.dropbear.id.au>

next in thread | previous in thread | raw e-mail | index | archive | help
At 9:26 AM +1100 1/31/03, Tim Robbins wrote:
>On Thu, Jan 30, 2003 at 06:35:24AM -0700, M. Warner Losh wrote:
>
>>  In message: <200301301200.h0UC0Qfd081080@repoman.freebsd.org>
>>              "Tim J. Robbins" <tjr@FreeBSD.org> writes:
>>  : tjr         2003/01/30 04:00:26 PST
>>  :
>>  :   Modified files:
>>  :     lib/libc/stdio       gets.c
>>  :   Log:
>>  :   Remove runtime warning about gets().
>>
>>  Why?
>
>Because it's tacky, redundant (we already warn at compile time)
>and not worth the space. NetBSD and BSD/OS removed it years ago.
>
>If you disagree and would like me to back this out, tell me.
>I don't feel strongly enough about this to argue.

When someone does a 'portupgrade -a', how many compile-time warnings
do you think they actually read?  If they're doing binary (package)
upgrades, they don't even have the chance to see compile-time warnings.

I'd say that the runtime warning is fine.  I'm quite comfortable
with it being tacky, as long as it does not get in the way of
the program actually working.  That's just my opinion though.  I
would prefer the warning return, but I'm not rabid about it.

-- 
Garance Alistair Drosehn            =   gad@gilead.netel.rpi.edu
Senior Systems Programmer           or  gad@freebsd.org
Rensselaer Polytechnic Institute    or  drosih@rpi.edu

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?p05200f3bba5f6bea34cf>