[Bf-committers] The big warning hunt patch
jgilbert at tigr.org
Tue Mar 8 21:16:28 CET 2005
@@ -507,14 +507,14 @@
GraphicsImportComponent theImporter = NULL;
- Str255 dst;
+// Str255 dst; /*unused*/
in quicktime_import.c breaks the windows compile. The variable dst is
not used on apple but is used on windows:
err = FSPathMakeRef(theFullPath, &myRef, 0);
err = FSGetCatalogInfo(&myRef, kFSCatInfoNone, NULL, NULL,
err = FSMakeFSSpec(0, 0L, dst, &theFSSpec);
Renderconverter now imports 2 files that are from other projects so the
project files will need to add:
to their compiler paths or else BL_imbuf will not compile.
Other than that everything else seems ok.
Note that msvc7 compiler has different warnings entirely and different
warnings are ouput.
From: bf-committers-bounces at projects.blender.org
[mailto:bf-committers-bounces at projects.blender.org] On Behalf Of
Sent: Monday, March 07, 2005 3:19 PM
To: bf-blender developers
Subject: [Bf-committers] The big warning hunt patch
the definitive (?) patch is here :
Please check if all goes well, i will commit it in 48H if nobody veto
I'm down to 48 warnings on my XCode build system with the following
-Wall -Wno-subscripts -Wmissing-prototypes for C files.
-Wall -Wno-subscripts C++ files.
other build systems may yield different results due to other options
Note that the level 2 C++ : -Wold-style-cast has been disabled in the
makefiles as it was giving thousands of warnings which is useless.
We should find and define on each platform an equivalent set of
meaningfull warnings to which each new commit should succeed, this
would keep the code clean.
On a side note, scons way of managing the includes should be corrected,
it is bad to give 7 times the same path ;)
Bf-committers mailing list
Bf-committers at projects.blender.org
More information about the Bf-committers