[Bf-committers] The big warning hunt patch

[Martin Poirier]

Good idea, I did the same thing:

http://www.elysiun.com/~theeth/W1.txt
http://www.elysiun.com/~theeth/W2.txt
http://www.elysiun.com/~theeth/W3.txt
http://www.elysiun.com/~theeth/W4.txt

Like you said, most are casts (double -> float, int ->
short), some signed/unsigned and unused variables.

Applying the patch went well, just had to add the
quicktime #ifdef WIN32 thingy and some include paths
to the renderconverter project and all was good to go.
I'll commit the projectfile changes after the patch is
commited.

Martin

--- "Gilbert, Joseph" <jgilbert at tigr.org> wrote:
> I guess if anyone is interested I built the win32
> release from msvc7 with the warning patch on various
> warning lvls. Msvc7 defines 4 warning lvls (and a
> suppress warnings lvl). Maybe this would help with
> the 'appropriate' warning lvls :-)
>  
> Build logs:
> http://www.paposo.com/warnings/WarningLVL1.txt
> http://www.paposo.com/warnings/WarningLVL2.txt
> http://www.paposo.com/warnings/WarningLVL3.txt
> http://www.paposo.com/warnings/WarningLVL4.txt
>  
> Many of msvc's warnings are simply casting and
> unused variable warnings.
>  
>  
> -----Original Message-----
> From: bf-committers-bounces at projects.blender.org
> [mailto:bf-committers-bounces at projects.blender.org]
> On Behalf Of Jean-Luc Peuriere
> Sent: Tuesday, March 08, 2005 3:33 PM
> To: bf-blender developers
> Subject: Re: [Bf-committers] The big warning hunt
> patch
>  
>  
> Le 8 mars 05, à 21:16, Gilbert, Joseph a écrit : 
> 	 
> 	@@ -507,14 +507,14 @@ 
> 	GraphicsImportComponent theImporter = NULL; 
> 	 
> 	FSSpec theFSSpec; 
> 	- Str255 dst; 
> 	+// Str255 dst; /*unused*/ 
> 	char theFullPath[255]; 
> 	 
>  
> Thanks. corrected in the future commit (protected by
> #ifdef _WIN32) 
> 	 
> 	Other than that everything else seems ok. 
>  
> nice. 
> 	 
> 	Note that msvc7 compiler has different warnings
> entirely and different 
> 	warnings are ouput. 
>  
> I know, but we should be able to define a reasonable
> equivalent set 
>  
> -- 
> Jean-Luc 
> > _______________________________________________
> Bf-committers mailing list
> Bf-committers at projects.blender.org
>
http://projects.blender.org/mailman/listinfo/bf-committers
> 

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com