[Bf-committers] The big warning hunt patch
theeth at yahoo.com
Wed Mar 9 02:24:03 CET 2005
Good idea, I did the same thing:
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
--- "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:
> 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
> 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;
> Thanks. corrected in the future commit (protected by
> #ifdef _WIN32)
> Other than that everything else seems ok.
> 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
> > _______________________________________________
> Bf-committers mailing list
> Bf-committers at projects.blender.org
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
More information about the Bf-committers