Re: [Flightgear-devel] FlightGear prerelease; RPMs available + PATCH

2007-11-28 Thread Ladislav Michnovič
2007/11/27, Andy Ross [EMAIL PROTECTED]:
  So it's only missing #include which should
  be in the code. See the attachments.
 
  [...]
 
   #include math.h
  +#include stdlib.h
  +#include cstring

 Surely that should be string.h, no?  It's just a style thing, but if
 you're modifying code that is already using ANSI C headers, and not
 Standard C++ headers, you should stay with the existing convention.
 It's especially weird to add one of each. :)

O.K. Thanks for your hint.
 Ladislav.

 Andy

 -
 This SF.net email is sponsored by: Microsoft
 Defy all challenges. Microsoft(R) Visual Studio 2005.
 http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
 ___
 Flightgear-devel mailing list
 Flightgear-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/flightgear-devel


-
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
___
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel


Re: [Flightgear-devel] FlightGear prerelease; RPMs available + PATCH

2007-11-27 Thread Ladislav Michnovič
Hi.

 I have yet another two patches to get FlightGear and Simgear
compiling with new gcc 4.3, which is stricter about missing
declarations of functions. So it's only missing #include which should
be in the code. See the attachments.

Btw. I'm building the RPMs for Fedora, Mandriva and SuSE in Build
Service (http://build.opensuse.org). So you can easily install them,
searching the FlightGear on the web page
http://software.opensuse.org/search and choose the home:lmich
repository.

 Regards Ladislav.
--- src/Airports/parking.cxx
+++ src/Airports/parking.cxx
@@ -27,6 +27,7 @@
 #endif
 
 //#include algorithm
+#include stdlib.h
 
 #include simgear/compiler.h
 
--- src/Airports/runwayprefs.cxx
+++ src/Airports/runwayprefs.cxx
@@ -26,6 +26,8 @@
 #endif
 
 #include math.h
+#include stdlib.h
+#include cstring
 //#include algorithm
 
 #include simgear/compiler.h
--- src/Airports/runways.cxx
+++ src/Airports/runways.cxx
@@ -27,6 +27,7 @@
 
 #include math.h   // fabs()
 #include stdio.h  // sprintf()
+#include stdlib.h //atoi()
 
 #include simgear/compiler.h
 #include simgear/debug/logstream.hxx
--- src/FDM/JSBSim/input_output/FGfdmSocket.cpp
+++ src/FDM/JSBSim/input_output/FGfdmSocket.cpp
@@ -37,6 +37,7 @@
 INCLUDES
 %%*/
 
+#include cstring
 #include FGfdmSocket.h
 
 namespace JSBSim {
--- src/FDM/YASim/yasim-test.cpp
+++ src/FDM/YASim/yasim-test.cpp
@@ -1,4 +1,6 @@
 #include stdio.h
+#include stdlib.h
+#include cstring
 
 #include simgear/props/props.hxx
 #include simgear/xml/easyxml.hxx
--- src/Main/util.cxx
+++ src/Main/util.cxx
@@ -21,6 +21,7 @@
 #include simgear/compiler.h
 
 #include math.h
+#include stdlib.h
 
 #include vector
 SG_USING_STD(vector);
--- BUILD/SimGear-0.3.11-pre2/simgear/io/sg_file.cxx
+++ BUILD/SimGear-0.3.11-pre2/simgear/io/sg_file.cxx
@@ -21,6 +21,8 @@
 // $Id: sg_file.cxx,v 1.8 2006-03-08 18:16:08 mfranz Exp $
 
 
+#include cstring
+
 #include simgear/compiler.h
 
 #include STL_STRING
--- BUILD/SimGear-0.3.11-pre2/simgear/io/sg_serial.cxx
+++ BUILD/SimGear-0.3.11-pre2/simgear/io/sg_serial.cxx
@@ -20,6 +20,8 @@
 //
 // $Id: sg_serial.cxx,v 1.6 2006-03-08 18:16:08 mfranz Exp $
 
+#include stdlib.h
+#include cstring
 
 #include simgear/compiler.h
 
--- BUILD/SimGear-0.3.11-pre2/simgear/misc/strutils.cxx
+++ BUILD/SimGear-0.3.11-pre2/simgear/misc/strutils.cxx
@@ -21,6 +21,7 @@
 // $Id: strutils.cxx,v 1.3 2006-03-08 18:16:08 mfranz Exp $
 
 #include ctype.h
+#include cstring
 #include strutils.hxx
 
 namespace simgear {
--- BUILD/SimGear-0.3.11-pre2/simgear/misc/tabbed_values.cxx
+++ BUILD/SimGear-0.3.11-pre2/simgear/misc/tabbed_values.cxx
@@ -20,6 +20,8 @@
 //
 // $Id: tabbed_values.cxx,v 1.3 2006-03-08 18:16:08 mfranz Exp $
 
+#include stdlib.h
+
 #include tabbed_values.hxx
 
 #include assert.h
--- BUILD/SimGear-0.3.11-pre2/simgear/screen/RenderTexture.cpp
+++ BUILD/SimGear-0.3.11-pre2/simgear/screen/RenderTexture.cpp
@@ -46,6 +46,7 @@
  * Nov. 2005, Use the simgear logging facility, Erik Hofman
  * Mar. 2006, Add MAC OS X support, Alexander Powell
  */
+#include cstring
 
 #ifdef HAVE_CONFIG_H
 #  include simgear_config.h
--- BUILD/SimGear-0.3.11-pre2/simgear/screen/TestRenderTexture.cpp
+++ BUILD/SimGear-0.3.11-pre2/simgear/screen/TestRenderTexture.cpp
@@ -16,6 +16,7 @@
 
 #include assert.h
 #include stdio.h
+#include stdlib.h
 
 void Reshape(int w, int h);
 
--- BUILD/SimGear-0.3.11-pre2/simgear/screen/shader.cpp
+++ BUILD/SimGear-0.3.11-pre2/simgear/screen/shader.cpp
@@ -18,6 +18,9 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 
+#include cstring
+#include stdlib.h
+
 #ifdef HAVE_CONFIG_H
 #  include simgear_config.h
 #endif 
-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/___
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel


Re: [Flightgear-devel] FlightGear prerelease; RPMs available + PATCH

2007-11-27 Thread Durk Talsma
On Tuesday 27 November 2007 15:22, Ladislav Michnovič wrote:
 Hi.

  I have yet another two patches to get FlightGear and Simgear
 compiling with new gcc 4.3, which is stricter about missing
 declarations of functions. So it's only missing #include which should
 be in the code. See the attachments.

Hi Ladislav,

Committed.

Thanks for the patches.

Cheers,
Durk

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel


Re: [Flightgear-devel] FlightGear prerelease; RPMs available + PATCH

2007-11-27 Thread Andy Ross
 So it's only missing #include which should
 be in the code. See the attachments.

 [...]

  #include math.h
 +#include stdlib.h
 +#include cstring

Surely that should be string.h, no?  It's just a style thing, but if
you're modifying code that is already using ANSI C headers, and not
Standard C++ headers, you should stay with the existing convention.
It's especially weird to add one of each. :)

Andy

-
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse012070mrt/direct/01/
___
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel