bug-commoncpp
[Top][All Lists]
Advanced

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

Re: Compiler warnings; minor thread.h patch


From: Federico Montesino Pouzols
Subject: Re: Compiler warnings; minor thread.h patch
Date: Mon, 30 Dec 2002 13:44:16 +0100
User-agent: Mutt/1.4i

        Ok, the patch has been just committed to CVS. I have added an
AM_CONDITIONAL so that these compiling options are only used when
compiling with GCC.

        For now, these options are not used for the library sources of
the RELEASE1 branch, since we have yet to sort out the issue with long
long. I have also looked at what happens when you specify -W and other
subtle options, and it seems there are more details to fix...

On Mon, Dec 30, 2002 at 11:46:27AM +0200, Albert Strasheim wrote:
> Hello,
> 
> Attached is a slightly modified and extended patch for compiler warnings 
> seen with gcc 3.2 (some also seen with gcc 2.96).
> 
> On Sun, 29 Dec 2002, Albert Strasheim wrote:
> 
> > Attached is a patch to fix all the warnings (when compiling with 
> > -Wall, -ansi and -pedantic) in src, demos, and tests, with the 
> > exception of an usused variable in one of the demos and the long long 
> > integer constant warnings in sha.cpp (not quite sure what needs to be 
> > done here).
> 
> The new patch lists all possible cases in switches, instead of using a 
> default case to suppress the warning. This is probably more correct.
>  
> > I'll feed cc++ to gcc 3.2 and the Intel C++ compiler when I get back to 
> > work during the second week in January.
> 
> Compiled with gcc 3.2. Fixed some more warnings.
>  
> Cheers,
> 
> Albert




reply via email to

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