[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Discuss-gnuradio] Re: [Commit-gnuradio] r10410 -gnuradio/trunk/gnur

From: Don Ward
Subject: Re: [Discuss-gnuradio] Re: [Commit-gnuradio] r10410 -gnuradio/trunk/gnuradio-core/src/lib/missing
Date: Tue, 10 Feb 2009 10:11:13 -0500

Michael Dickens wrote:

Fix missing header include (Don Ward)

Modified: gnuradio/trunk/gnuradio-core/src/lib/missing/
#include "posix_memalign.h"
+#include <malloc.h>

This change breaks compile on OSX ... malloc.h is in /usr/include/
malloc/ on OSX, and is not needed on OSX or LINUX to be best of my
knowledge for this code to compile ... only for the Windows
environment.  Here's my suggested patch ... does no harm on Linux,
takes care of the issue on Windows, and allows OSX to compile:

Oops, this is my mistake. I should have seen that valloc() is supposed to be in <stdlib.h> (but isn't there in Cygwin). Thanks for fixing this.

I'm worried, though, that valloc isn't available at all on MinGW. Is there another way to accomplish what is needed?

-- Don W.

Index: configure.ac
--- configure.ac (revision 10414)
+++ configure.ac (working copy)
@@ -127,7 +127,7 @@
 AC_CHECK_HEADERS(linux/ppdev.h dev/ppbus/ppi.h sys/mman.h sys/
select.h sys/types.h)
 AC_CHECK_HEADERS(sys/resource.h stdint.h sched.h signal.h sys/
+AC_CHECK_HEADERS(windows.h malloc.h)

 dnl Checks for typedefs, structures, and compiler characteristics.
Index: gnuradio-core/src/lib/missing/posix_memalign.cc
--- gnuradio-core/src/lib/missing/posix_memalign.cc (revision 10414)
+++ gnuradio-core/src/lib/missing/posix_memalign.cc (working copy)
@@ -25,7 +25,10 @@

 #include "posix_memalign.h"
 #include <malloc.h>


Discuss-gnuradio mailing list


No virus found in this incoming message.
Checked by AVG - www.avg.com
Version: 8.0.233 / Virus Database: 270.10.20/1943 - Release Date: 02/07/09 13:39:00

reply via email to

[Prev in Thread] Current Thread [Next in Thread]