Archived

This forum has been archived. Please start a new discussion on GitHub.

Compilation fixes for GCC 4.3

GCC 4.3 fails to compile Ice 3.2, IceE 1.1 and IceE-translators 1.1 due to missing includes. This bug was first reported against Ice Debian packages. The original messages may be found at:

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=417787

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=417790

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=417803

The following patch by Martin Michlmayr should solve these issues
--- zeroc-ice-3.2.0/src/IceUtil/Exception.cpp   2007-02-01 17:40:28.000000000 +0100
+++ zeroc-ice-3.2.0.new/src/IceUtil/Exception.cpp       2007-04-08 20:33:46.000000000 +0200
@@ -8,6 +8,7 @@
 // **********************************************************************

 #include <IceUtil/Exception.h>
+#include <cstdlib>

 using namespace std;

--- zeroc-icee-translators-1.1.0/src/IceUtil/Exception.cpp      2006-02-21 02:55:09.000000000 +0100
+++ zeroc-icee-translators-1.1.0.new/src/IceUtil/Exception.cpp  2007-04-08 21:05:18.000000000 +0200
@@ -8,6 +8,7 @@
 // **********************************************************************

 #include <IceUtil/Exception.h>
+#include <cstdlib>

 using namespace std;

--- zeroc-icee-1.1.0/include/IceE/Buffer.h      2006-03-07 19:51:01.000000000 +0100
+++ zeroc-icee-1.1.0.new/include/IceE/Buffer.h  2007-04-08 20:43:03.000000000 +0200
@@ -11,6 +11,7 @@
 #define ICEE_BUFFER_H

 #include <IceE/Config.h>
+#include <cstdlib>

 #define ICE_SMALL_MESSAGE_BUFFER_OPTIMIZATION
 #define ICE_BUFFER_FIXED_SIZE 64

Comments

  • One hunk was missing

    Sorry, I missed this one for IceE 1.1.0:
    --- test/IceE/custom/MyByteSeq.cpp~	2007-04-04 15:01:46.000000000 +0000
    +++ test/IceE/custom/MyByteSeq.cpp	2007-04-04 15:01:53.000000000 +0000
    @@ -8,6 +8,7 @@
     // **********************************************************************
     
     #include <MyByteSeq.h>
    +#include <cstdlib>
     
     MyByteSeq::MyByteSeq()
         : _size(0),
    
  • bernard
    bernard Jupiter, FL
    Hi Francisco,

    Thanks for the patches; we'll add these include <cstdlib> in the next release.

    Bernard