warning fixes
authorCampbell Barton <ideasman42@gmail.com>
Sat, 3 Sep 2011 15:36:36 +0000 (15:36 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 3 Sep 2011 15:36:36 +0000 (15:36 +0000)
intern/iksolver/intern/IK_QSegment.cpp
source/blender/blenkernel/intern/mesh_validate.c
source/blender/collada/DocumentExporter.cpp
source/blender/collada/ExtraTags.cpp
source/blender/makesrna/intern/rna_scene_api.c

index df4fbc8..ba4fbb8 100644 (file)
@@ -319,7 +319,7 @@ void IK_QSegment::RemoveChild(IK_QSegment *child)
        else {
                IK_QSegment *seg = m_child;
 
-               while (seg->m_sibling != child);
+               while (seg->m_sibling != child)
                        seg = seg->m_sibling;
 
                if (child == seg->m_sibling)
index 9c916d5..7039859 100644 (file)
@@ -166,7 +166,7 @@ int BKE_mesh_validate_arrays(Mesh *me, MVert *UNUSED(mverts), unsigned int totve
                }
 
                if(BLI_edgehash_haskey(edge_hash, med->v1, med->v2)) {
-                       PRINT("    edge %u: is a duplicate of, %u\n", i, GET_INT_FROM_POINTER(BLI_edgehash_lookup(edge_hash, med->v1, med->v2)));
+                       PRINT("    edge %u: is a duplicate of, %d\n", i, GET_INT_FROM_POINTER(BLI_edgehash_lookup(edge_hash, med->v1, med->v2)));
                        remove= do_fixes;
                }
 
index 285ab28..6e78088 100644 (file)
@@ -328,9 +328,7 @@ void DocumentExporter::exportCurrentScene(Scene *sce, const char* filename, bool
        //scale = RNA_struct_find_property(&unit_settings, "scale_length");
 
        std::string unitname = "meter";
-       float linearmeasure = 1.0f;
-
-       linearmeasure = RNA_float_get(&unit_settings, "scale_length");
+       float linearmeasure = RNA_float_get(&unit_settings, "scale_length");
 
        switch(RNA_property_enum_get(&unit_settings, system)) {
                case USER_UNIT_NONE:
@@ -368,8 +366,7 @@ void DocumentExporter::exportCurrentScene(Scene *sce, const char* filename, bool
 
        asset.setUnit(unitname, linearmeasure);
        asset.setUpAxisType(COLLADASW::Asset::Z_UP);
-       // TODO: need an Author field in userpref
-       if(strlen(U.author) > 0) {
+       if(U.author[0] != '\0') {
                asset.getContributor().mAuthor = U.author;
        }
        else {
index 653d4a3..f0c6d22 100644 (file)
@@ -90,32 +90,28 @@ std::string ExtraTags::asString( std::string tag, bool *ok)
 void ExtraTags::setData(std::string tag, short *data)
 {
        bool ok = false;
-       int tmp = 0;
-       tmp = asInt(tag, &ok);
+       int tmp = asInt(tag, &ok);
        if(ok)
                *data = (short)tmp;
 }
 void ExtraTags::setData(std::string tag, int *data)
 {
        bool ok = false;
-       int tmp = 0;
-       tmp = asInt(tag, &ok);
+       int tmp = asInt(tag, &ok);
        if(ok)
                *data = tmp;
 }
 void ExtraTags::setData(std::string tag, float *data)
 {
        bool ok = false;
-       float tmp = 0.0f;
-       tmp = asFloat(tag, &ok);
+       float tmp = asFloat(tag, &ok);
        if(ok)
                *data = tmp;
 }
 void ExtraTags::setData(std::string tag, char *data)
 {
        bool ok = false;
-       int tmp = 0;
-       tmp = asInt(tag, &ok);
+       int tmp = asInt(tag, &ok);
        if(ok)
                *data = (char)tmp;
 }
index 1220c4f..fd7987c 100644 (file)
@@ -111,7 +111,7 @@ void RNA_api_scene(StructRNA *srna)
 #ifdef WITH_COLLADA
        /* don't remove this, as COLLADA exporting cannot be done through operators in render() callback. */
        func= RNA_def_function(srna, "collada_export", "rna_Scene_collada_export");
-       parm= RNA_def_string(func, "filepath", "", FILE_MAX, "File Path", "File path to write Collada file.");
+       RNA_def_string(func, "filepath", "", FILE_MAX, "File Path", "File path to write Collada file.");
        parm= RNA_def_boolean(func, "selected", 0, "Export only selected", "Export only selected elements.");
        RNA_def_property_flag(parm, PROP_REQUIRED);
        RNA_def_property_subtype(parm, PROP_FILEPATH); /* allow non utf8 */