Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Oct 2001 08:06:27 -0800 (PST)
From:      John Baldwin <jhb@FreeBSD.org>
To:        Peter Pentchev <roam@ringlet.net>
Cc:        cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, Garrett Wollman <wollman@khavrinen.lcs.mit.edu>
Subject:   Re: cvs commit: src/include malloc.h
Message-ID:  <XFMail.011030080627.jhb@FreeBSD.org>
In-Reply-To: <20011030164640.C31154@straylight.oblivion.bg>

next in thread | previous in thread | raw e-mail | index | archive | help

On 30-Oct-01 Peter Pentchev wrote:
> On Tue, Oct 30, 2001 at 10:57:17AM -0500, Garrett Wollman wrote:
>> <<On Tue, 30 Oct 2001 16:11:05 +0200, Peter Pentchev <roam@ringlet.net>
>> said:
>> 
>> > Just BTW, what is the correct test to use for malloc.h vs stdlib.h?
>> > Will __STDC__ be enough?
>> 
>> #if HAVE_STDLIB_H
> 
> Eep.. :)
> 
>> __STDC__ tells you about the compiler; it doesn't tell you whether the
>> compilation environment is actually valid.  In FreeBSD, we have always
>> had <stdlib.h>, even when compiling with `gcc -traditional'.
> 
> Well, I figured that much.. but this does not help me much :)
> 
> I'm asking more from the standpoint of submitting fixes back to authors
> of various ported software; e.g. right now, I'm struggling with a big
> piece of software which has a *lot* of:
> 
>#if defined(BSDI)
>#include <malloc.h>
>#else
>#include <stdlib.h>
>#endif
> 
> I could easily change that to defined(BSDI) || defined(__FreeBSD__),
> but.. you mean there really is no way to do this better? :\

Err, that code is right.  malloc.h is deprecated and stdlib.h should be what is
included.

-- 

John Baldwin <jhb@FreeBSD.org> -- http://www.FreeBSD.org/~jhb/
PGP Key: http://www.baldwin.cx/~john/pgpkey.asc
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/

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?XFMail.011030080627.jhb>