Received: by taz.hyperreal.com (8.6.12/8.6.5) id MAA01733; Wed, 24 Jul 1996 12:03:41 -0700 Received: from acidik.organic.com by taz.hyperreal.com (8.6.12/8.6.5) with ESMTP id MAA01727; Wed, 24 Jul 1996 12:03:39 -0700 Received: from localhost (akosut@localhost) by acidik.organic.com (8.7.5/8.6.12) with SMTP id MAA00554 for ; Wed, 24 Jul 1996 12:03:15 -0700 (PDT) X-Authentication-Warning: acidik.organic.com: akosut owned process doing -bs Date: Wed, 24 Jul 1996 12:03:14 -0700 (PDT) From: Alexei Kosut To: new-httpd@hyperreal.com Subject: Re: malloc patch In-Reply-To: <9607240847.AA09619@ jrc.it> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-new-httpd@apache.org Precedence: bulk Reply-To: new-httpd@hyperreal.com On Wed, 24 Jul 1996, Dirk vanGulik wrote: > Has some-one committed something like this, which > plugs a malloc() fail centrally ? +1 on the patch, except for one thing: > + * > + * Dirk.vanGulik@jrc.it - Added some code which makes it barf when > + * it cannot malloc memory; this saves a lot > + * of checks further down in the code. Sorry, but no. -1 Do not pass go, do not collect $200. RST gets his name there because he wrote the original code, so does Rob M if he wrote it. We also put names in some of the modules people wrote or totall rewrote on their own, like mod_status or mod_actions or mod_imap. But if you get your name in the code for a seven line patch, then I demand my name in all the peices of code *I've* written. And so will everyone else. I just don't think this is appropriate. CVS keeps logs, we don't need them in the files. There have been a lot of modifications to alloc.c since version 0.8.0 of Apache, when it was introduced, like there have been to all the files. None of them got attribution, neither do yours. A little thing, but IMHO it's important. -- Alexei Kosut The Apache HTTP Server http://www.nueva.pvt.k12.ca.us/~akosut/ http://www.apache.org/