code cleanup: remove paranoid/invalid NULL checks and also reduce some unneeded size_...
authorCampbell Barton <ideasman42@gmail.com>
Sat, 15 Sep 2012 07:31:17 +0000 (07:31 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 15 Sep 2012 07:31:17 +0000 (07:31 +0000)
12 files changed:
source/blender/blenfont/intern/blf_font.c
source/blender/blenfont/intern/blf_internal.h
source/blender/blenfont/intern/blf_lang.c
source/blender/blenkernel/intern/object.c
source/blender/blenkernel/intern/text.c
source/blender/editors/armature/meshlaplacian.c
source/blender/editors/armature/poseobject.c
source/blender/editors/interface/view2d.c
source/blender/gpu/intern/gpu_draw.c
source/blender/imbuf/intern/IMB_metadata.h
source/blender/imbuf/intern/metadata.c
source/blender/makesdna/DNA_meshdata_types.h

index ab43804db8e6be039000c7c5431971c520a02ef3..cb05c89921ebfe05f95b2dced88c7e88198ded9a 100644 (file)
@@ -159,7 +159,7 @@ static void blf_font_ensure_ascii_table(FontBLF *font)
        }                                                                            \
 } (void)0
 
-void blf_font_draw(FontBLF *font, const char *str, unsigned int len)
+void blf_font_draw(FontBLF *font, const char *str, size_t len)
 {
        unsigned int c;
        GlyphBLF *g, *g_prev = NULL;
@@ -191,7 +191,7 @@ void blf_font_draw(FontBLF *font, const char *str, unsigned int len)
 }
 
 /* faster version of blf_font_draw, ascii only for view dimensions */
-void blf_font_draw_ascii(FontBLF *font, const char *str, unsigned int len)
+void blf_font_draw_ascii(FontBLF *font, const char *str, size_t len)
 {
        unsigned char c;
        GlyphBLF *g, *g_prev = NULL;
index ba66fce8374b077ff23204b88de4f965940c965b..65a547839788cdbb6b02389feb3b196e76ddc2f7 100644 (file)
@@ -51,8 +51,8 @@ struct FontBLF *blf_font_new_from_mem(const char *name, const unsigned char *mem
 void blf_font_attach_from_mem(struct FontBLF *font, const unsigned char *mem, int mem_size);
 
 void blf_font_size(struct FontBLF *font, int size, int dpi);
-void blf_font_draw(struct FontBLF *font, const char *str, unsigned int len);
-void blf_font_draw_ascii(struct FontBLF *font, const char *str, unsigned int len);
+void blf_font_draw(struct FontBLF *font, const char *str, size_t len);
+void blf_font_draw_ascii(struct FontBLF *font, const char *str, size_t len);
 void blf_font_buffer(struct FontBLF *font, const char *str);
 void blf_font_boundbox(struct FontBLF *font, const char *str, struct rctf *box);
 void blf_font_width_and_height(struct FontBLF *font, const char *str, float *width, float *height);
index 2b7e225d709e3a2612499323649ff8491e43954a..d0b772221144f28e14e514324a59881239963721 100644 (file)
@@ -129,7 +129,7 @@ void BLF_lang_init(void)
 }
 
 /* get LANG/LANGUAGE environment variable */
-static void get_language_variable(const char *varname, char *var, int maxlen)
+static void get_language_variable(const char *varname, char *var, const size_t maxlen)
 {
        char *env = getenv(varname);
 
@@ -149,7 +149,7 @@ static void get_language_variable(const char *varname, char *var, int maxlen)
 /* get language to be used based on locale(which might be empty when using default language) and
  * LANG environment variable
  */
-static void get_language(const char *locale, const char *lang, char *language, int maxlen)
+static void get_language(const char *locale, const char *lang, char *language, const size_t maxlen)
 {
        if (locale[0]) {
                BLI_strncpy(language, locale, maxlen);
index 4fb235ac9f7e4bee890565ad0ae1aaca5a7f3067..5ba56a85c724468987449e6b9f5a0fecf7e6658f 100644 (file)
@@ -3288,7 +3288,7 @@ void BKE_object_groups_clear(Scene *scene, Base *base, Object *object)
 {
        Group *group = NULL;
 
-       BLI_assert(base->object == object);
+       BLI_assert((base == NULL) || (base->object == object));
 
        if (scene && base == NULL) {
                base = BKE_scene_base_find(scene, object);
index af5a8f5d59db9379b38f7a68236f5754e62c378c..3c89fdba13a58b9df84501f468a54e857cfa8816 100644 (file)
@@ -3070,7 +3070,6 @@ int setcurr_tab_spaces(Text *text, int space)
        const char *comm = "#";
        const char indent = (text->flags & TXT_TABSTOSPACES) ? ' ' : '\t';
        static const char *back_words[] = {"return", "break", "continue", "pass", "yield", NULL};
-       if (!text) return 0;
        if (!text->curl) return 0;
 
        while (text->curl->line[i] == indent) {
index 6b6d2a1505f0659ae58e9f6823de7c3d6c59cca5..346ed0002bd9fb92037bafeabbe16689f62d0231 100644 (file)
@@ -667,7 +667,7 @@ void heat_bone_weighting(Object *ob, Mesh *me, float (*verts)[3], int numsource,
        *err_str = NULL;
 
        /* count triangles and create mask */
-       if ((use_face_sel = (me->editflag & ME_EDIT_PAINT_MASK) != 0) ||
+       if ((use_face_sel = ((me->editflag & ME_EDIT_PAINT_MASK) != 0)) ||
            (use_vert_sel = ((me->editflag & ME_EDIT_VERT_SEL) != 0)))
        {
                mask = MEM_callocN(sizeof(int) * me->totvert, "heat_bone_weighting mask");
index b1a613bef5d82b4ef8a76b27ca467e3e02c0330b..ac1766aff699b35a8d2b597664c5c42175128204 100644 (file)
@@ -728,7 +728,7 @@ static int pose_select_grouped_exec(bContext *C, wmOperator *op)
        short changed = 0;
        
        /* sanity check */
-       if (ELEM(NULL, ob, ob->pose))
+       if (ob->pose == NULL)
                return OPERATOR_CANCELLED;
                
        /* selection types 
index 0f1d1a24dcbc8fed275eac079912bdcfd688a5ff..9c71746d0452ed5a315402172c3d951e117fed3f 100644 (file)
@@ -851,7 +851,7 @@ void UI_view2d_totRect_set_resize(View2D *v2d, int width, int height, int resize
        if (scroll & V2D_SCROLL_VERTICAL) 
                height -= V2D_SCROLL_HEIGHT;
        
-       if (ELEM3(0, v2d, width, height)) {
+       if (ELEM(0, width, height)) {
                if (G.debug & G_DEBUG)
                        printf("Error: View2D totRect set exiting: v2d=%p width=%d height=%d\n", (void *)v2d, width, height);  // XXX temp debug info
                return;
index 4314a784511c784c7b67c57dbbe75e5dfc2610f0..ab3ab0a133e1881cdf365869bc080be48a35205a 100644 (file)
@@ -1221,7 +1221,7 @@ void GPU_begin_object_materials(View3D *v3d, RegionView3D *rv3d, Scene *scene, O
         * - object transparency off: for glsl we draw both in a single pass, and
         * for solid we don't use transparency at all. */
        GMS.use_alpha_pass = (do_alpha_after != NULL);
-       GMS.is_alpha_pass = (v3d && v3d->transp);
+       GMS.is_alpha_pass = (v3d->transp != FALSE);
        if (GMS.use_alpha_pass)
                *do_alpha_after = FALSE;
        
index 4f512e04ef8287a59ddd65521ee7c04f4318530d..a68d7a7813e800f3a272977d1328160284fed339 100644 (file)
@@ -62,7 +62,7 @@ void IMB_metadata_free(struct ImBuf *img);
  *  \param len - length of value buffer allocated by user.
  *  \return    - 1 (true) if ImageInfo present and value for the key found, 0 (false) otherwise
  */
-int IMB_metadata_get_field(struct ImBuf *img, const char *key, char *value, int len);
+int IMB_metadata_get_field(struct ImBuf *img, const char *key, char *value, const size_t len);
 
 /** set user data in the ImMetaData struct, which has to be allocated with IMB_metadata_create
  *  before calling this function.
index 1ce2053fde589a711750fcb0c66cded4455a00ae..fed55c05eea04d0e6d082143a9e120807d6cfad1 100644 (file)
@@ -63,7 +63,7 @@ void IMB_metadata_free(struct ImBuf *img)
        }
 }
 
-int IMB_metadata_get_field(struct ImBuf *img, const char *key, char *field, int len)
+int IMB_metadata_get_field(struct ImBuf *img, const char *key, char *field, const size_t len)
 {
        ImMetaData *info;
        int retval = 0;
index 0b826070fe4b5ff14aaa7042f2044f1d8da2a0b3..54e1e762f8b06cf71f6cacb263ba64fb619dc638 100644 (file)
@@ -119,7 +119,8 @@ typedef struct MLoopUV {
 #define MLOOPUV_PINNED 4
 
 /* at the moment alpha is abused for vertex painting
- * and not used for transparency, note that red and blue are swapped */
+ * and not used for transparency,
+ * note that red and blue are _not_ swapped, as they are with #MCol */
 typedef struct MLoopCol {
        char r, g, b, a;
 } MLoopCol;