tidy up
authorCampbell Barton <ideasman42@gmail.com>
Tue, 20 Sep 2011 04:38:59 +0000 (04:38 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 20 Sep 2011 04:38:59 +0000 (04:38 +0000)
- quiet some warnings
- set some functions static
- replace materialbyname with generic BLI_findstring call

source/blender/blenkernel/intern/material.c
source/blender/editors/armature/editarmature.c
source/blender/editors/sound/sound_ops.c
source/blender/editors/space_view3d/view3d_select.c
source/blender/nodes/composite/node_composite_util.h
source/blender/render/intern/source/rendercore.c

index 60ba5654f77376c266375aa00268201a2faa063a..6b8d486ae56e5636e78d3d8299b63bc1fbd05981 100644 (file)
@@ -37,6 +37,7 @@
 
 #include <string.h>
 #include <math.h>
+#include <stddef.h>
 
 #include "MEM_guardedalloc.h"
 
@@ -1480,7 +1481,7 @@ void paste_matcopybuf(Material *ma)
 
 /*********************** texface to material convert functions **********************/
 /* encode all the TF information into a single int */
-int encode_tfaceflag(MTFace *tf, int convertall)
+static int encode_tfaceflag(MTFace *tf, int convertall)
 {
        /* calculate the flag */
        int flag = tf->mode;
@@ -1507,7 +1508,7 @@ int encode_tfaceflag(MTFace *tf, int convertall)
 }
 
 /* set the material options based in the tface flag */
-void decode_tfaceflag(Material *ma, int flag, int convertall)
+static void decode_tfaceflag(Material *ma, int flag, int convertall)
 {
        int alphablend; 
        GameSettings *game= &ma->game;
@@ -1544,7 +1545,7 @@ void decode_tfaceflag(Material *ma, int flag, int convertall)
 }
 
 /* boolean check to see if the mesh needs a material */
-int check_tfaceneedmaterial(int flag)
+static int check_tfaceneedmaterial(int flag)
 {
        // check if the flags we have are not deprecated != than default material options
        // also if only flags are visible and collision see if all objects using this mesh have this option in physics
@@ -1575,7 +1576,7 @@ int check_tfaceneedmaterial(int flag)
 
 /* return number of digits of an integer */
 // XXX to be optmized or replaced by an equivalent blender internal function
-int integer_getdigits(int number)
+static int integer_getdigits(int number)
 {
        int i=0;
        if (number == 0) return 1;
@@ -1587,7 +1588,7 @@ int integer_getdigits(int number)
        return i;
 }
 
-void calculate_tface_materialname(char *matname, char *newname, int flag)
+static void calculate_tface_materialname(char *matname, char *newname, int flag)
 {
        // if flag has only light and collision and material matches those values
        // you can do strcpy(name, mat_name);
@@ -1599,7 +1600,7 @@ void calculate_tface_materialname(char *matname, char *newname, int flag)
 }
 
 /* returns -1 if no match */
-int mesh_getmaterialnumber(Mesh *me, Material *ma) {
+static int mesh_getmaterialnumber(Mesh *me, Material *ma) {
        int a;
 
        for (a=0; a<me->totcol; a++)
@@ -1610,7 +1611,7 @@ int mesh_getmaterialnumber(Mesh *me, Material *ma) {
 }
 
 /* append material */
-int mesh_addmaterial(Mesh *me, Material *ma)
+static int mesh_addmaterial(Mesh *me, Material *ma)
 {
        material_append_id(&me->id, NULL);
        me->mat[me->totcol-1]= ma;
@@ -1620,17 +1621,7 @@ int mesh_addmaterial(Mesh *me, Material *ma)
        return me->totcol-1;
 }
 
-Material *materialbyname(Main *main, char *name)
-{
-       Material *mat = NULL;
-       for (mat=main->mat.first;mat;mat=mat->id.next) {
-               if (strcmp(mat->id.name, name)==0)
-                       return mat; 
-       }
-       return NULL;
-}
-
-void set_facetexture_flags(Material *ma, Image *image)
+static void set_facetexture_flags(Material *ma, Image *image)
 {
        if(image) {
                ma->mode |= MA_FACETEXTURE;
@@ -1642,17 +1633,17 @@ void set_facetexture_flags(Material *ma, Image *image)
 }
 
 /* returns material number */
-int convert_tfacenomaterial(Main *main, Mesh *me, MTFace *tf, int flag)
+static int convert_tfacenomaterial(Main *main, Mesh *me, MTFace *tf, int flag)
 {
        Material *ma;
-       char name[MAX_ID_NAME];
+       char idname[MAX_ID_NAME];
        int mat_nr= -1;
        
        /* new material, the name uses the flag*/
-       sprintf(name, "MAMaterial.TF.%0*d", integer_getdigits(flag), flag);
+       sprintf(idname, "MAMaterial.TF.%0*d", integer_getdigits(flag), flag);
        
-       if ((ma=materialbyname(main, name))) {
-               mat_nr = mesh_getmaterialnumber(me, ma);
+       if ((ma= BLI_findstring(&main->mat, idname+2, offsetof(ID, name)+2))) {
+               mat_nr= mesh_getmaterialnumber(me, ma);
                /* assign the material to the mesh */
                if(mat_nr == -1) mat_nr= mesh_addmaterial(me, ma);
 
@@ -1661,10 +1652,10 @@ int convert_tfacenomaterial(Main *main, Mesh *me, MTFace *tf, int flag)
        }
        /* create a new material */
        else {
-               ma=add_material(name+2);
+               ma= add_material(idname+2);
 
                if(ma){
-                       printf("TexFace Convert: Material \"%s\" created.\n", name+2);
+                       printf("TexFace Convert: Material \"%s\" created.\n", idname+2);
                        mat_nr= mesh_addmaterial(me, ma);
                        
                        /* if needed set "Face Textures [Alpha]" Material options */
@@ -1677,7 +1668,7 @@ int convert_tfacenomaterial(Main *main, Mesh *me, MTFace *tf, int flag)
                        ma->game.flag = -flag;
                        id_us_min((ID *)ma);    
                }
-               else printf("Error: Unable to create Material \"%s\" for Mesh \"%s\".", name+2, me->id.name+2);
+               else printf("Error: Unable to create Material \"%s\" for Mesh \"%s\".", idname+2, me->id.name+2);
        }
 
        /* set as converted, no need to go bad to this face */
@@ -1686,7 +1677,7 @@ int convert_tfacenomaterial(Main *main, Mesh *me, MTFace *tf, int flag)
 }
 
 /* Function to fully convert materials */
-void convert_tfacematerial(Main *main, Material *ma)
+static void convert_tfacematerial(Main *main, Material *ma)
 {
        Mesh *me;
        Material *mat_new;
@@ -1695,7 +1686,7 @@ void convert_tfacematerial(Main *main, Material *ma)
        int flag, index;
        int a, mat_nr;
        CustomDataLayer *cdl;
-       char name[MAX_ID_NAME];
+       char idname[MAX_ID_NAME];
 
        for(me=main->mesh.first; me; me=me->id.next){
                /* check if this mesh uses this material */
@@ -1719,9 +1710,9 @@ void convert_tfacematerial(Main *main, Material *ma)
                        flag = encode_tfaceflag(tf, 1);
 
                        /* the name of the new material */
-                       calculate_tface_materialname(ma->id.name, (char *)&name, flag);
+                       calculate_tface_materialname(ma->id.name, (char *)&idname, flag);
 
-                       if ((mat_new=materialbyname(main, name))) {
+                       if ((mat_new= BLI_findstring(&main->mat, idname+2, offsetof(ID, name)+2))) {
                                /* material already existent, see if the mesh has it */
                                mat_nr = mesh_getmaterialnumber(me, mat_new);
                                /* material is not in the mesh, add it */
@@ -1732,14 +1723,14 @@ void convert_tfacematerial(Main *main, Material *ma)
                                mat_new=copy_material(ma);
                                if(mat_new){
                                        /* rename the material*/
-                                       strcpy(mat_new->id.name, name);
+                                       strcpy(mat_new->id.name, idname);
                                        id_us_min((ID *)mat_new);       
 
                                        mat_nr= mesh_addmaterial(me, mat_new);
                                        decode_tfaceflag(mat_new, flag, 1);
                                }
                                else {
-                                       printf("Error: Unable to create Material \"%s\" for Mesh \"%s.", name+2, me->id.name+2);
+                                       printf("Error: Unable to create Material \"%s\" for Mesh \"%s.", idname+2, me->id.name+2);
                                        mat_nr = mf->mat_nr;
                                        continue;
                                }
index fbe336b3493c773f3f9bd9ccf3e50c8a127790e1..566ff09c366223988470dfdeb5f4789ed0eb1645 100644 (file)
@@ -4298,7 +4298,7 @@ int ED_do_pose_selectbuffer(Scene *scene, Base *base, unsigned int *buffer, shor
                 * note, special exception for armature mode so we can do multi-select
                 * we could check for multi-select explicitly but think its fine to
                 * always give pradictable behavior in weight paint mode - campbell */
-               if (!(extend) || (base->object != ob_act && !(ob_act->mode & OB_MODE_WEIGHT_PAINT))) {
+               if (!(extend) || ((ob_act && ob_act->mode & OB_MODE_WEIGHT_PAINT) == 0)) {
                        ED_pose_deselectall(ob, 0);
                        nearBone->flag |= (BONE_SELECTED|BONE_TIPSEL|BONE_ROOTSEL);
                        arm->act_bone= nearBone;
index 539e975095ea0b266acd804571af996927568635..5b72e87f95a56eda1ee13deb07a07b0d493d6174 100644 (file)
@@ -436,7 +436,7 @@ static void mixdown_draw(bContext *C, wmOperator *op)
 }
 #endif // WITH_AUDASPACE
 
-void SOUND_OT_mixdown(wmOperatorType *ot)
+static void SOUND_OT_mixdown(wmOperatorType *ot)
 {
 #ifdef WITH_AUDASPACE
        static EnumPropertyItem format_items[] = {
index aa3166a7a65cce8047f734db69ef31df5a33b57d..d2c7ae337298ec8dbb3628180fc650c42d1ea7ea 100644 (file)
@@ -1976,17 +1976,18 @@ static int vertsel_vert_pick(struct bContext *C, Mesh *me, const int mval[2], un
 
 /* mouse selection in weight paint */
 /* gets called via generic mouse select operator */
-int mouse_wp_select(bContext *C, const int mval[2], short extend, Object *obact, Mesh* me)
+static int mouse_weight_paint_vertex_select(bContext *C, const int mval[2], short extend, Object *obact)
 {
+       Mesh* me= obact->data; /* already checked for NULL */
        unsigned int index = 0;
        MVert *mv;
        if(vertsel_vert_pick(C, me, mval, &index, 1)) {
                mv = me->mvert+index;
                if(extend) {
-                       mv->flag ^= 1;
+                       mv->flag ^= SELECT;
                } else {
                        paintvert_deselect_all_visible(obact, SEL_DESELECT, FALSE);
-                       mv->flag |= 1;
+                       mv->flag |= SELECT;
                }
                paintvert_flush_flags(obact);
                WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obact->data);
@@ -2039,12 +2040,10 @@ static int view3d_select_invoke(bContext *C, wmOperator *op, wmEvent *event)
                return PE_mouse_particles(C, event->mval, extend);
        else if(obact && paint_facesel_test(obact))
                retval = paintface_mouse_select(C, obact, event->mval, extend);
-       
-       else if (paint_vertsel_test(obact)) {
-               retval = mouse_wp_select(C, event->mval, extend, obact, obact->data);
-       } else {
+       else if (paint_vertsel_test(obact))
+               retval = mouse_weight_paint_vertex_select(C, event->mval, extend, obact);
+       else
                retval = mouse_select(C, event->mval, extend, center, enumerate);
-       }
 
        /* passthrough allows tweaks
         * FINISHED to signal one operator worked
index 57ebe2191ddec9a8961fc48fda415794fe464fdb..f3e0f811f136d5fafe7f0d194a75405bedfbfbf9 100644 (file)
 #include "RE_shader_ext.h"
 #include "RE_render_ext.h"
 
+/* only for forward declarations */
+#include "NOD_composite.h"
+
+
 /* *************************** operations support *************************** */
 
 /* general signal that's in output sockets, and goes over the wires */
index 59a505a3195ca3e9691dc5848640786c9a82cd4f..654ec7cc927d389aab4ecbe95f4ae8e87290d22c 100644 (file)
@@ -992,7 +992,7 @@ static void convert_to_key_alpha(RenderPart *pa, RenderLayer *rl)
 }
 
 /* adds only alpha values */
-void edge_enhance_tile(RenderPart *pa, float *rectf, int *rectz)
+static void edge_enhance_tile(RenderPart *pa, float *rectf, int *rectz)
 {
        /* use zbuffer to define edges, add it to the image */
        int y, x, col, *rz, *rz1, *rz2, *rz3;
@@ -1133,7 +1133,7 @@ typedef struct ZbufSolidData {
        float *edgerect;
 } ZbufSolidData;
 
-void make_pixelstructs(RenderPart *pa, ZSpan *zspan, int sample, void *data)
+static void make_pixelstructs(RenderPart *pa, ZSpan *zspan, int sample, void *data)
 {
        ZbufSolidData *sdata= (ZbufSolidData*)data;
        ListBase *lb= sdata->psmlist;