On Fri, Aug 27, 2010 at 2:11 AM, luxInteg <[email protected]> wrote: > On Thursday 26 August 2010, 06:20:57 Nathan Letwory wrote > >>I'll be committing a patch by Imran Syed (freakabcd). This has also a >>small change that will need a newer opencollada revision - latest should >>work fine. > > > Thanks a lot > > I managed to get hold of revision 31592. On a pure 64-bit (amd64)-linux > machine the throttled cmake compile output looks like so: > > ################################################################ > Scanning dependencies of target bf_collada > [ 84%] Building CXX object > source/blender/collada/CMakeFiles/bf_collada.dir/DocumentImporter.cpp.o > In file included from > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:34: > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h: In > constructor 'COLLADAFW::ColorOrTexture::ColorOrTexture(COLLADAFW::Color)': > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:39: > warning: 'COLLADAFW::ColorOrTexture::mColor' will be initialized after > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:38: > warning: 'COLLADAFW::ColorOrTexture::Type COLLADAFW::ColorOrTexture::mType' > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:46: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h: In > constructor 'COLLADAFW::ColorOrTexture::ColorOrTexture(COLLADAFW::Texture)': > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:40: > warning: 'COLLADAFW::ColorOrTexture::mTexture' will be initialized after > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:38: > warning: 'COLLADAFW::ColorOrTexture::Type COLLADAFW::ColorOrTexture::mType' > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWColorOrTexture.h:55: > warning: when initialized here > In file included from > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWEffectCommon.h:18, > from > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWEffect.h:19, > from > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:35: > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h: In > constructor 'COLLADAFW::FloatOrParam::FloatOrParam(float)': > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:51: > warning: 'COLLADAFW::FloatOrParam::mFloatValue' will be initialized after > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:47: > warning: 'COLLADAFW::FloatOrParam::Type COLLADAFW::FloatOrParam::mType' > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:60: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h: In > constructor 'COLLADAFW::FloatOrParam::FloatOrParam(const COLLADAFW::Param&)': > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:55: > warning: 'COLLADAFW::FloatOrParam::mParam' will be initialized after > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:47: > warning: 'COLLADAFW::FloatOrParam::Type COLLADAFW::FloatOrParam::mType' > $INSTALLDIR/opencollada/COLLADAFramework/include/COLLADAFWFloatOrParam.h:66: > warning: when initialized here > In file included from > $INSTALLDIR/opencollada/COLLADASaxFrameworkLoader/include/COLLADASaxFWLLoader.h:17, > from > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:60: > $INSTALLDIR/opencollada/COLLADASaxFrameworkLoader/include/COLLADASaxFWLKinematicsIntermediateData.h: > In constructor > 'COLLADASaxFWL::KinematicsNewParam::KinematicsNewParam(COLLADASaxFWL::KinematicsNewParam::ValueType)': > $INSTALLDIR/opencollada/COLLADASaxFrameworkLoader/include/COLLADASaxFWLKinematicsIntermediateData.h:237: > warning: enumeration value 'VALUETYPE_UNKNOWN' not handled in switch > $INSTALLDIR/opencollada/COLLADASaxFrameworkLoader/include/COLLADASaxFWLKinematicsIntermediateData.h: > In member function 'void > COLLADASaxFWL::KinematicsFloatOrParam::setFloatValue(float)': > $INSTALLDIR/opencollada/COLLADASaxFrameworkLoader/include/COLLADASaxFWLKinematicsIntermediateData.h:626: > warning: statement has no effect > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp: In member > function 'void > AnimationImporter::animation_to_fcurves(COLLADAFW::AnimationCurve*)': > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2196: warning: > format '%d' expects type 'int', but argument 3 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp: In destructor > 'AnimationImporter::~AnimationImporter()': > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2287: warning: > format '%u' expects type 'unsigned int', but argument 3 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp: In member > function 'Object* AnimationImporter::translate_animation(COLLADAFW::Node*, > std::map<COLLADAFW::UniqueId, Object*, std::less<COLLADAFW::UniqueId>, > std::allocator<std::pair<const COLLADAFW::UniqueId, Object*> > >&, > std::map<COLLADAFW::UniqueId, COLLADAFW::Node*, > std::less<COLLADAFW::UniqueId>, std::allocator<std::pair<const > COLLADAFW::UniqueId, COLLADAFW::Node*> > >&, > COLLADAFW::Transformation::TransformationType, Object*)': > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2630: warning: > format '%u' expects type 'unsigned int', but argument 4 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp: In member > function 'bool > AnimationImporter::evaluate_animation(COLLADAFW::Transformation*, float (*) > [4], float, const char*)': > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2968: warning: > format '%u' expects type 'unsigned int', but argument 3 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2990: warning: > format '%u' expects type 'unsigned int', but argument 4 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:2992: warning: > format '%u' expects type 'unsigned int', but argument 4 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:3019: warning: > format '%u' expects type 'unsigned int', but argument 4 has type 'size_t' > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp: In member > function 'void MeshImporter::read_faces(COLLADAFW::Mesh*, Mesh*, int)': > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[0]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[0]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[0]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[0]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'a[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'a[0]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[2]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[2]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[1]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[1]' was declared here > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: warning: > 'b[0]' may be used uninitialized in this function > $SOURCES/blender/source/blender/collada/DocumentImporter.cpp:1866: note: > 'b[0]' was declared here > [ 84%] Building CXX object > source/blender/collada/CMakeFiles/bf_collada.dir/collada.cpp.o > [ 84%] Building CXX object > source/blender/collada/CMakeFiles/bf_collada.dir/DocumentExporter.cpp.o > In file included from > $SOURCES/blender/source/blender/collada/DocumentExporter.cpp:75: > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWNode.h: In > constructor 'COLLADASW::Node::Node(COLLADASW::StreamWriter*, bool)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWNode.h:56: > warning: 'COLLADASW::Node::mType' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWNode.h:44: > warning: 'COLLADASW::String COLLADASW::Node::mNodeId' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWNode.h:63: > warning: when initialized here > In file included from > $SOURCES/blender/source/blender/collada/DocumentExporter.cpp:80: > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWPrimitves.h: In > constructor > 'COLLADASW::PrimitivesBase::PrimitivesBase(COLLADASW::StreamWriter*, const > COLLADASW::String&)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWPrimitves.h:49: > warning: 'COLLADASW::PrimitivesBase::mInputList' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWPrimitves.h:40: > warning: 'COLLADASW::String COLLADASW::PrimitivesBase::mPrimitiveName' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWPrimitves.h:60: > warning: when initialized here > In file included from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWSampler.h:19, > from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWTexture.h:18, > from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:18, > from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWEffectProfile.h:18, > from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWLibraryEffects.h:16, > from > $SOURCES/blender/source/blender/collada/DocumentExporter.cpp:84: > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h: In > constructor 'COLLADASW::Annotation::Annotation(COLLADASW::StreamWriter*, const > COLLADASW::String&, const COLLADASW::ValueType::ColladaType&, const float*, > int)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:74: > warning: 'COLLADASW::Annotation::mNumValues' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:71: > warning: 'COLLADASW::Annotation::UnionType > COLLADASW::Annotation::mUnionType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:85: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h: In > constructor 'COLLADASW::Annotation::Annotation(COLLADASW::StreamWriter*, const > COLLADASW::String&, const COLLADASW::ValueType::ColladaType&, const bool*, > int)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:74: > warning: 'COLLADASW::Annotation::mNumValues' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:71: > warning: 'COLLADASW::Annotation::UnionType > COLLADASW::Annotation::mUnionType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:101: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h: In > constructor 'COLLADASW::Annotation::Annotation(COLLADASW::StreamWriter*, const > COLLADASW::String&, const COLLADASW::ValueType::ColladaType&, const int*, > int)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:74: > warning: 'COLLADASW::Annotation::mNumValues' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:71: > warning: 'COLLADASW::Annotation::UnionType > COLLADASW::Annotation::mUnionType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:117: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h: In > constructor 'COLLADASW::Annotation::Annotation(COLLADASW::StreamWriter*, const > COLLADASW::String&, const COLLADASW::ValueType::ColladaType&, const char*, > int)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:74: > warning: 'COLLADASW::Annotation::mNumValues' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:71: > warning: 'COLLADASW::Annotation::UnionType > COLLADASW::Annotation::mUnionType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWAnnotation.h:133: > warning: when initialized here > In file included from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWEffectProfile.h:18, > from > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWLibraryEffects.h:16, > from > $SOURCES/blender/source/blender/collada/DocumentExporter.cpp:84: > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h: > In constructor 'COLLADASW::ColorOrTexture::ColorOrTexture(COLLADASW::Color, > const COLLADASW::String&)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:39: > warning: 'COLLADASW::ColorOrTexture::mColor' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:38: > warning: 'COLLADASW::ColorOrTexture::Type COLLADASW::ColorOrTexture::mType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:47: > warning: when initialized here > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h: > In constructor 'COLLADASW::ColorOrTexture::ColorOrTexture(COLLADASW::Texture, > const COLLADASW::String&)': > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:40: > warning: 'COLLADASW::ColorOrTexture::mTexture' will be initialized after > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:38: > warning: 'COLLADASW::ColorOrTexture::Type COLLADASW::ColorOrTexture::mType' > $INSTALLDIR/opencollada/COLLADAStreamWriter/include/COLLADASWColorOrTexture.h:56: > warning: when initialized here > Linking CXX static library ../../../lib/libbf_collada.a > [ 84%] Built target bf_collada > ################################################################ > > when installed > > the blender executible is ~44 Mbytes > linteg [ ~/blenderOC1/bin ]# du blender > 44828 blender > linteg [ ~/blenderOC1/bin ]# > > without opencollada it is ~26 Mbytes > linteg [ ~/blenderPLAIN/bin ]# du blender > 26224 blender > linteg [ ~/blenderPLAIN/bin ]# > > --------------------------- > > thanks again
We used to have a lot of these warnings in the GameEngine but I modified the order just to get better build logs and to see real problems. >From my experience these warnings are harmless, would be good to get rid of these but not high priority IMHO. _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers
