Re: [boost] [logging] lots of warnings

From: Jens Seidel (jensseidel_at_[hidden])
Date: 2008-02-09 10:39:03

On Fri, Feb 08, 2008 at 07:24:21PM +0200, John Torjo wrote:
> Jens Seidel wrote:
> > John, please try to test your code before you commit. There are even some
> >   
> I always do that - unfortunately MSVC did not complain. gcc did not 
> complain either. I've just added -W -Wall so that I catch all these 
> "cool" warnings.

OK, but you probably test not all files. Have you tried my simple
boost_check script or did you wrote a own script to test *all* files?

It's hard for me to believe that you can use wrong namespaces, include
non-existing files, ... Maybe the compiler will indeed not complain if
the code is not instanciated, don't know.

PS: How about a mini Linux system in a virtual environment? There are
much more possibilities to create such an environment in Linux (chroot,
UML, qemu, xen, bochs, and many many more) but there should be a
solution also for your system.

PS2: Whouldn't it a good idea if Boost could allow developer access to
a Linux server? I strongly assume that e.g. the web pages are self hosted on
a Open Source platform.
 
> p.s. sorry about syslog - did not test it since I don't have linux.

No problem. It is really great that you nevertheless support it.
 
> Here's the latest compilation:
> 
> Compiler: gcc 4.1
> Building Makefile: 
> "D:\john\code\boost_sandbox\logging\libs\logging\internal\gcc\Makefile.win"
> Executing  make...
> make.exe -f 
> "D:\john\code\boost_sandbox\logging\libs\logging\internal\gcc\Makefile.win" 
> all
> g++.exe -c main.cpp -o main.o -I".."  -I"D:/boosts/boost_1_33_1"  
> -I"C:/mingw/include/c++/4.1.2/backward"  -I"C:/mingw/include/c++/4.1.2"  
> -I"C:/mingw/include/c++"  -I"C:/mingw/include"  -I"."  -I"../../../.."  
> -W -Wall 
 
Mmh, I assume it is jam stuff which I never used myself. I wonder about
 -I"C:/mingw/include/c++/4.1.2/backward" -I"C:/mingw/include/c++/4.1.2"  
 -I"C:/mingw/include/c++" -I"C:/mingw/include"  
shouldn't it be in the default compiler search path?

Maybe you should also add -pedantic (and -Wno-long-long if it complains
about long long?).

Jens