corrections for redundant null checks & transform printing a string into its self.
authorCampbell Barton <ideasman42@gmail.com>
Fri, 29 Apr 2011 06:59:18 +0000 (06:59 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 29 Apr 2011 06:59:18 +0000 (06:59 +0000)
source/blender/blenkernel/intern/particle.c
source/blender/blenkernel/intern/pointcache.c
source/blender/editors/curve/editcurve.c
source/blender/editors/interface/interface_draw.c
source/blender/editors/space_graph/graph_select.c
source/blender/editors/space_time/time_ops.c
source/blender/editors/transform/transform.c
source/blender/editors/transform/transform_snap.c

index 5b58372090530d8f028dd182a3028f1dff4be037..5ec4c36177f644d7dc5d79c6fb0d523e90bee789 100644 (file)
@@ -3030,7 +3030,7 @@ void psys_cache_paths(ParticleSimulationData *sim, float cfra)
 
        psys->totcached = totpart;
 
-       if(psys && psys->lattice){
+       if(psys->lattice){
                end_latt_deform(psys->lattice);
                psys->lattice= NULL;
        }
index 179aab064b40e486ad77513b06b8b9ea478ccd28..a7dd98ff404d5175b601ab7aed777ac6ba17251b 100644 (file)
@@ -1382,7 +1382,7 @@ static void ptcache_find_frames_around(PTCacheID *pid, unsigned int frame, int *
                        }
                }
 
-               if(pm && !pm2) {
+               if(!pm2) {
                        *fra1 = 0;
                        *fra2 = pm->frame;
                }
@@ -1844,7 +1844,8 @@ static int ptcache_write(PTCacheID *pid, int cfra, int overwrite)
        if(cache->flag & PTCACHE_DISK_CACHE) {
                error += !ptcache_mem_frame_to_disk(pid, pm);
 
-               if(pm) {
+               // if(pm) /* pm is always set */
+               {
                        ptcache_data_free(pm);
                        ptcache_extra_free(pm);
                        MEM_freeN(pm);
index 9bae4ea6569f4c09119ee4d5fef03b0f00baccfd..cb73f20fe7b54478d7e32a5504cd2453f08433ce 100644 (file)
@@ -1234,7 +1234,6 @@ void make_editNurb(Object *obedit)
        Nurb *nu, *newnu, *nu_act= NULL;
        KeyBlock *actkey;
 
-       if(obedit==NULL) return;
 
        set_actNurb(obedit, NULL);
 
@@ -1591,7 +1590,7 @@ static int deleteflagNurb(bContext *C, wmOperator *UNUSED(op), int flag)
        BPoint *bp, *bpn, *newbp;
        int a, b, newu, newv, sel;
 
-       if(obedit && obedit->type==OB_SURF);
+       if(obedit->type==OB_SURF);
        else return OPERATOR_CANCELLED;
 
        cu->lastsel= NULL;
index f8ce278814fa5a4ddd37f43a7b8dbca6447d4ff0..c9b5b9f284836fa86c6378184d61e7ce13bebd25 100644 (file)
@@ -765,8 +765,6 @@ void ui_draw_but_HISTOGRAM(ARegion *ar, uiBut *but, uiWidgetColors *UNUSED(wcol)
        //float alpha;
        GLint scissor[4];
        
-       if (hist==NULL) { printf("hist is null \n"); return; }
-       
        rect.xmin = (float)recti->xmin+1;
        rect.xmax = (float)recti->xmax-1;
        rect.ymin = (float)recti->ymin+SCOPE_RESIZE_PAD+2;
index 920f477e7ebe3424ca214654422400de80b7498b..6c713d6e469c273fa7bf3d5e00b51b80c726c3fa 100644 (file)
@@ -484,11 +484,8 @@ static void columnselect_graph_keys (bAnimContext *ac, short mode)
                 */
                for (ce= ked.list.first; ce; ce= ce->next) {
                        /* set frame for validation callback to refer to */
-                       if (ale)
-                               ked.f1= BKE_nla_tweakedit_remap(adt, ce->cfra, NLATIME_CONVERT_UNMAP);
-                       else
-                               ked.f1= ce->cfra;
-                       
+                       ked.f1= BKE_nla_tweakedit_remap(adt, ce->cfra, NLATIME_CONVERT_UNMAP);
+
                        /* select elements with frame number matching cfraelem */
                        ANIM_fcurve_keyframes_loop(&ked, ale->key_data, ok_cb, select_cb, NULL);
                }
index 34380fe4fd6dfe648afe0cabe3bd1d591a50860f..e9559426b81c7017367797e9ead8850f8f004008 100644 (file)
 #include "time_intern.h"
 
 /* ****************** Start/End Frame Operators *******************************/
-
 static int time_set_sfra_exec (bContext *C, wmOperator *UNUSED(op))
 {
        Scene *scene= CTX_data_scene(C);
-       int frame= CFRA;
-       
+       int frame;
+
        if (scene == NULL)
                return OPERATOR_CANCELLED;
-               
+
+       frame= CFRA;
        /* if 'end frame' (Preview Range or Actual) is less than 'frame', 
         * clamp 'frame' to 'end frame'
         */
@@ -93,11 +93,13 @@ static void TIME_OT_start_frame_set (wmOperatorType *ot)
 static int time_set_efra_exec (bContext *C, wmOperator *UNUSED(op))
 {
        Scene *scene= CTX_data_scene(C);
-       int frame= CFRA;
-       
+       int frame;
+
        if (scene == NULL)
                return OPERATOR_CANCELLED;
-               
+
+       frame= CFRA;
+
        /* if 'start frame' (Preview Range or Actual) is greater than 'frame', 
         * clamp 'frame' to 'end frame'
         */
index ae863efa9f79c35c8511749da84ad8aa73859475..5e227cba5232aa6ae852822ad3edf87a6ec531a4 100644 (file)
@@ -3151,7 +3151,7 @@ static void applyRotation(TransInfo *t, float angle, float axis[3])
 
 int Rotation(TransInfo *t, const short UNUSED(mval[2]))
 {
-       char str[128];
+       char str[128], *spos= str;
        
        float final;
 
@@ -3177,18 +3177,20 @@ int Rotation(TransInfo *t, const short UNUSED(mval[2]))
                
                outputNumInput(&(t->num), c);
                
-               sprintf(str, "Rot: %s %s %s", &c[0], t->con.text, t->proptext);
+               spos+= sprintf(spos, "Rot: %s %s %s", &c[0], t->con.text, t->proptext);
 
                /* Clamp between -180 and 180 */
                final= angle_wrap_rad(DEG2RADF(final));
        }
        else {
-               sprintf(str, "Rot: %.2f%s %s", RAD2DEGF(final), t->con.text, t->proptext);
+               spos += sprintf(spos, "Rot: %.2f%s %s", RAD2DEGF(final), t->con.text, t->proptext);
        }
        
-       if (t->flag & (T_PROP_EDIT|T_PROP_CONNECTED))
-               sprintf(str, "%s Proportional size: %.2f", str, t->prop_size);
-       
+       if (t->flag & (T_PROP_EDIT|T_PROP_CONNECTED)) {
+               spos += sprintf(spos, " Proportional size: %.2f", t->prop_size);
+       }
+       (void)spos;
+
        t->values[0] = final;
        
        applyRotation(t, final, t->axis);
@@ -3257,7 +3259,7 @@ static void applyTrackball(TransInfo *t, float axis1[3], float axis2[3], float a
 
 int Trackball(TransInfo *t, const short UNUSED(mval[2]))
 {
-       char str[128];
+       char str[128], *spos= str;
        float axis1[3], axis2[3];
        float mat[3][3], totmat[3][3], smat[3][3];
        float phi[2];
@@ -3281,14 +3283,19 @@ int Trackball(TransInfo *t, const short UNUSED(mval[2]))
 
                outputNumInput(&(t->num), c);
 
-               sprintf(str, "Trackball: %s %s %s", &c[0], &c[20], t->proptext);
+               spos += sprintf(spos, "Trackball: %s %s %s", &c[0], &c[20], t->proptext);
 
                phi[0] = DEG2RADF(phi[0]);
                phi[1] = DEG2RADF(phi[1]);
        }
        else {
-               sprintf(str, "Trackball: %.2f %.2f %s", RAD2DEGF(phi[0]), RAD2DEGF(phi[1]), t->proptext);
+               spos += sprintf(spos, "Trackball: %.2f %.2f %s", RAD2DEGF(phi[0]), RAD2DEGF(phi[1]), t->proptext);
+       }
+
+       if (t->flag & (T_PROP_EDIT|T_PROP_CONNECTED)) {
+               spos += sprintf(spos, " Proportional size: %.2f", t->prop_size);
        }
+       (void)spos;
 
        vec_rot_to_mat3( smat,axis1, phi[0]);
        vec_rot_to_mat3( totmat,axis2, phi[1]);
@@ -3345,6 +3352,7 @@ void initTranslation(TransInfo *t)
 }
 
 static void headerTranslation(TransInfo *t, float vec[3], char *str) {
+       char *spos= str;
        char tvec[60];
        char distvec[20];
        char autoik[20];
@@ -3395,24 +3403,26 @@ static void headerTranslation(TransInfo *t, float vec[3], char *str) {
        if (t->con.mode & CON_APPLY) {
                switch(t->num.idx_max) {
                case 0:
-                       sprintf(str, "D: %s (%s)%s %s  %s", &tvec[0], distvec, t->con.text, t->proptext, &autoik[0]);
+                       spos += sprintf(spos, "D: %s (%s)%s %s  %s", &tvec[0], distvec, t->con.text, t->proptext, &autoik[0]);
                        break;
                case 1:
-                       sprintf(str, "D: %s   D: %s (%s)%s %s  %s", &tvec[0], &tvec[20], distvec, t->con.text, t->proptext, &autoik[0]);
+                       spos += sprintf(spos, "D: %s   D: %s (%s)%s %s  %s", &tvec[0], &tvec[20], distvec, t->con.text, t->proptext, &autoik[0]);
                        break;
                case 2:
-                       sprintf(str, "D: %s   D: %s  D: %s (%s)%s %s  %s", &tvec[0], &tvec[20], &tvec[40], distvec, t->con.text, t->proptext, &autoik[0]);
+                       spos += sprintf(spos, "D: %s   D: %s  D: %s (%s)%s %s  %s", &tvec[0], &tvec[20], &tvec[40], distvec, t->con.text, t->proptext, &autoik[0]);
                }
        }
        else {
                if(t->flag & T_2D_EDIT)
-                       sprintf(str, "Dx: %s   Dy: %s (%s)%s %s", &tvec[0], &tvec[20], distvec, t->con.text, t->proptext);
+                       spos += sprintf(spos, "Dx: %s   Dy: %s (%s)%s %s", &tvec[0], &tvec[20], distvec, t->con.text, t->proptext);
                else
-                       sprintf(str, "Dx: %s   Dy: %s  Dz: %s (%s)%s %s  %s", &tvec[0], &tvec[20], &tvec[40], distvec, t->con.text, t->proptext, &autoik[0]);
+                       spos += sprintf(spos, "Dx: %s   Dy: %s  Dz: %s (%s)%s %s  %s", &tvec[0], &tvec[20], &tvec[40], distvec, t->con.text, t->proptext, &autoik[0]);
        }
        
-       if (t->flag & (T_PROP_EDIT|T_PROP_CONNECTED))
-               sprintf(str, "%s Proportional size: %.2f", str, t->prop_size);
+       if (t->flag & (T_PROP_EDIT|T_PROP_CONNECTED)) {
+               spos += sprintf(spos, " Proportional size: %.2f", t->prop_size);
+       }
+       (void)spos;
 }
 
 static void applyTranslation(TransInfo *t, float vec[3]) {
index 022e09b3da401bd5121768bf3a9eebbaf02fbf98..789a262f0755aba45d00fe685a184c034eb022dd 100644 (file)
@@ -1670,7 +1670,7 @@ static void removeDoublesPeel(ListBase *depth_peels)
        {
                DepthPeel *next_peel = peel->next;
                
-               if (peel && next_peel && ABS(peel->depth - next_peel->depth) < 0.0015f)
+               if (next_peel && ABS(peel->depth - next_peel->depth) < 0.0015f)
                {
                        peel->next = next_peel->next;