Revision: 55999
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=55999
Author:   campbellbarton
Date:     2013-04-13 00:43:49 +0000 (Sat, 13 Apr 2013)
Log Message:
-----------
code cleanup: warnings + style

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/BKE_paint.h
    trunk/blender/source/blender/blenkernel/intern/paint.c
    trunk/blender/source/blender/editors/screen/glutil.c
    trunk/blender/source/blender/editors/sculpt_paint/paint_cursor.c
    trunk/blender/source/blender/editors/sculpt_paint/paint_utils.c
    trunk/blender/source/blender/gpu/intern/gpu_extensions.c
    trunk/blender/source/blender/makesdna/DNA_scene_types.h
    trunk/blender/source/blender/makesrna/intern/rna_texture.c

Modified: trunk/blender/source/blender/blenkernel/BKE_paint.h
===================================================================
--- trunk/blender/source/blender/blenkernel/BKE_paint.h 2013-04-12 23:55:11 UTC 
(rev 55998)
+++ trunk/blender/source/blender/blenkernel/BKE_paint.h 2013-04-13 00:43:49 UTC 
(rev 55999)
@@ -72,7 +72,7 @@
 #define PAINT_INVALID_OVERLAY_TEXTURE_SECONDARY 2
 #define PAINT_INVALID_OVERLAY_CURVE 4
 
-void BKE_paint_invalidate_overlay_tex (struct Scene *scene,const struct Tex 
*tex);
+void BKE_paint_invalidate_overlay_tex (struct Scene *scene, const struct Tex 
*tex);
 void BKE_paint_invalidate_cursor_overlay (struct Scene *scene, struct 
CurveMapping *curve);
 void BKE_paint_invalidate_overlay_all(void);
 int BKE_paint_get_overlay_flags (void);

Modified: trunk/blender/source/blender/blenkernel/intern/paint.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/paint.c      2013-04-12 
23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/blenkernel/intern/paint.c      2013-04-13 
00:43:49 UTC (rev 55999)
@@ -29,8 +29,9 @@
  *  \ingroup bke
  */
 
+#include <stdlib.h>
+#include <string.h>
 
-
 #include "DNA_object_types.h"
 #include "DNA_mesh_types.h"
 #include "DNA_meshdata_types.h"
@@ -51,9 +52,6 @@
 
 #include "bmesh.h"
 
-#include <stdlib.h>
-#include <string.h>
-
 const char PAINT_CURSOR_SCULPT[3] = {255, 100, 100};
 const char PAINT_CURSOR_VERTEX_PAINT[3] = {255, 255, 255};
 const char PAINT_CURSOR_WEIGHT_PAINT[3] = {200, 200, 255};
@@ -81,18 +79,20 @@
                overlay_flags |= PAINT_INVALID_OVERLAY_CURVE;
 }
 
-void BKE_paint_invalidate_overlay_all()
+void BKE_paint_invalidate_overlay_all(void)
 {
-       overlay_flags |= PAINT_INVALID_OVERLAY_TEXTURE_SECONDARY;
-       overlay_flags |= PAINT_INVALID_OVERLAY_TEXTURE_PRIMARY;
-       overlay_flags |= PAINT_INVALID_OVERLAY_CURVE;
+       overlay_flags |= (PAINT_INVALID_OVERLAY_TEXTURE_SECONDARY |
+                         PAINT_INVALID_OVERLAY_TEXTURE_PRIMARY |
+                         PAINT_INVALID_OVERLAY_CURVE);
 }
 
-int BKE_paint_get_overlay_flags () {
+int BKE_paint_get_overlay_flags(void)
+{
        return overlay_flags;
 }
 
-void BKE_paint_reset_overlay_invalid (void) {
+void BKE_paint_reset_overlay_invalid(void)
+{
        overlay_flags &= ~(PAINT_INVALID_OVERLAY_TEXTURE_PRIMARY |
                           PAINT_INVALID_OVERLAY_TEXTURE_SECONDARY |
                           PAINT_INVALID_OVERLAY_CURVE);

Modified: trunk/blender/source/blender/editors/screen/glutil.c
===================================================================
--- trunk/blender/source/blender/editors/screen/glutil.c        2013-04-12 
23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/editors/screen/glutil.c        2013-04-13 
00:43:49 UTC (rev 55999)
@@ -1043,10 +1043,10 @@
                return;
 
        /* Dithering is not supported on GLSL yet */
-       force_fallback = ibuf->dither != 0.0f;
+       // force_fallback = ibuf->dither != 0.0f;
 
        /* Single channel images could not be transformed using GLSL yet */
-       force_fallback = ibuf->channels == 1;
+       // force_fallback = ibuf->channels == 1;
 
        /* If user decided not to use GLSL, fallback to glaDrawPixelsAuto */
        force_fallback = !ELEM(U.image_draw_method,

Modified: trunk/blender/source/blender/editors/sculpt_paint/paint_cursor.c
===================================================================
--- trunk/blender/source/blender/editors/sculpt_paint/paint_cursor.c    
2013-04-12 23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/editors/sculpt_paint/paint_cursor.c    
2013-04-13 00:43:49 UTC (rev 55999)
@@ -88,7 +88,7 @@
                //(mtex->brush_map_mode != MTEX_MAP_MODE_VIEW ||
                //(BKE_brush_size_get(vc->scene, brush) <= 
snap->BKE_brush_size_get)) &&
 
-                       (mtex->brush_map_mode != MTEX_MAP_MODE_TILED ||
+               (mtex->brush_map_mode != MTEX_MAP_MODE_TILED ||
                (vc->ar->winx == snap->winx &&
                vc->ar->winy == snap->winy))
                );
@@ -230,8 +230,8 @@
                                                        
paint_get_tex_pixel_col(&br->mtex, x, y, rgba, pool);
 
                                                if (br->mtex.brush_map_mode == 
MTEX_MAP_MODE_VIEW) {
-                                                       float curve_str = 
BKE_brush_curve_strength(br, len, 1);
-                                                       CLAMP(curve_str, 0.0, 
1.0);
+                                                       float curve_str = 
BKE_brush_curve_strength(br, len, 1.0f);
+                                                       CLAMP(curve_str, 0.0f, 
1.0f);
                                                        mul_v4_fl(rgba, 
curve_str);  /* Falloff curve */
                                                }
                                                buffer[index * 4]     = rgba[0] 
* 255;
@@ -245,10 +245,10 @@
                                                avg += br->texture_sample_bias;
 
                                                if (br->mtex.brush_map_mode == 
MTEX_MAP_MODE_VIEW)
-                                                       avg *= 
BKE_brush_curve_strength(br, len, 1);  /* Falloff curve */
+                                                       avg *= 
BKE_brush_curve_strength(br, len, 1.0f);  /* Falloff curve */
 
                                                /* clamp to avoid precision 
overflow */
-                                               CLAMP(avg, 0.0, 1.0);
+                                               CLAMP(avg, 0.0f, 1.0f);
                                                buffer[index] = 255 - 
(GLubyte)(255 * avg);
                                        }
                                }

Modified: trunk/blender/source/blender/editors/sculpt_paint/paint_utils.c
===================================================================
--- trunk/blender/source/blender/editors/sculpt_paint/paint_utils.c     
2013-04-12 23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/editors/sculpt_paint/paint_utils.c     
2013-04-13 00:43:49 UTC (rev 55999)
@@ -206,10 +206,10 @@
                rgba[2] = intensity;
                rgba[3] = 1.0f;
        }
-       CLAMP(rgba[0], 0.0, 1.0);
-       CLAMP(rgba[1], 0.0, 1.0);
-       CLAMP(rgba[2], 0.0, 1.0);
-       CLAMP(rgba[3], 0.0, 1.0);
+       CLAMP(rgba[0], 0.0f, 1.0f);
+       CLAMP(rgba[1], 0.0f, 1.0f);
+       CLAMP(rgba[2], 0.0f, 1.0f);
+       CLAMP(rgba[3], 0.0f, 1.0f);
 }
 
 /* 3D Paint */

Modified: trunk/blender/source/blender/gpu/intern/gpu_extensions.c
===================================================================
--- trunk/blender/source/blender/gpu/intern/gpu_extensions.c    2013-04-12 
23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/gpu/intern/gpu_extensions.c    2013-04-13 
00:43:49 UTC (rev 55999)
@@ -108,7 +108,7 @@
        GG.extdisabled = 1;
 }
 
-int GPU_max_texture_size ()
+int GPU_max_texture_size(void)
 {
        return GG.maxtexsize;
 }

Modified: trunk/blender/source/blender/makesdna/DNA_scene_types.h
===================================================================
--- trunk/blender/source/blender/makesdna/DNA_scene_types.h     2013-04-12 
23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/makesdna/DNA_scene_types.h     2013-04-13 
00:43:49 UTC (rev 55999)
@@ -920,18 +920,18 @@
 } UnifiedPaintSettings;
 
 typedef enum {
-       UNIFIED_PAINT_SIZE  = (1<<0),
-       UNIFIED_PAINT_ALPHA = (1<<1),
-       UNIFIED_PAINT_WEIGHT = (1<<5),
+       UNIFIED_PAINT_SIZE  = (1 << 0),
+       UNIFIED_PAINT_ALPHA = (1 << 1),
+       UNIFIED_PAINT_WEIGHT = (1 << 5),
 
        /* only used if unified size is enabled, mirros the brush flags
         * BRUSH_LOCK_SIZE and BRUSH_SIZE_PRESSURE */
-       UNIFIED_PAINT_BRUSH_LOCK_SIZE = (1<<2),
-       UNIFIED_PAINT_BRUSH_SIZE_PRESSURE   = (1<<3),
+       UNIFIED_PAINT_BRUSH_LOCK_SIZE = (1 << 2),
+       UNIFIED_PAINT_BRUSH_SIZE_PRESSURE   = (1 << 3),
 
        /* only used if unified alpha is enabled, mirrors the brush flag
         * BRUSH_ALPHA_PRESSURE */
-       UNIFIED_PAINT_BRUSH_ALPHA_PRESSURE  = (1<<4)
+       UNIFIED_PAINT_BRUSH_ALPHA_PRESSURE  = (1 << 4)
 } UnifiedPaintSettingsFlags;
 
 /* *************************************************************** */
@@ -1508,24 +1508,24 @@
 /* Sculpt.flags */
 /* These can eventually be moved to paint flags? */
 typedef enum SculptFlags {
-       SCULPT_SYMM_X = (1<<0),
-       SCULPT_SYMM_Y = (1<<1),
-       SCULPT_SYMM_Z = (1<<2),
-       SCULPT_LOCK_X = (1<<3),
-       SCULPT_LOCK_Y = (1<<4),
-       SCULPT_LOCK_Z = (1<<5),
-       SCULPT_SYMMETRY_FEATHER = (1<<6),
-       SCULPT_USE_OPENMP = (1<<7),
-       SCULPT_ONLY_DEFORM = (1<<8),
-       SCULPT_SHOW_DIFFUSE = (1<<9),
+       SCULPT_SYMM_X = (1 << 0),
+       SCULPT_SYMM_Y = (1 << 1),
+       SCULPT_SYMM_Z = (1 << 2),
+       SCULPT_LOCK_X = (1 << 3),
+       SCULPT_LOCK_Y = (1 << 4),
+       SCULPT_LOCK_Z = (1 << 5),
+       SCULPT_SYMMETRY_FEATHER = (1 << 6),
+       SCULPT_USE_OPENMP = (1 << 7),
+       SCULPT_ONLY_DEFORM = (1 << 8),
+       SCULPT_SHOW_DIFFUSE = (1 << 9),
 
        /* If set, the mesh will be drawn with smooth-shading in
         * dynamic-topology mode */
-       SCULPT_DYNTOPO_SMOOTH_SHADING = (1<<10),
+       SCULPT_DYNTOPO_SMOOTH_SHADING = (1 << 10),
 
        /* If set, dynamic-topology brushes will collapse short edges in
         * addition to subdividing long ones */
-       SCULPT_DYNTOPO_COLLAPSE = (1<<11)
+       SCULPT_DYNTOPO_COLLAPSE = (1 << 11)
 } SculptFlags;
 
 /* ImagePaintSettings.flag */

Modified: trunk/blender/source/blender/makesrna/intern/rna_texture.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_texture.c  2013-04-12 
23:55:11 UTC (rev 55998)
+++ trunk/blender/source/blender/makesrna/intern/rna_texture.c  2013-04-13 
00:43:49 UTC (rev 55999)
@@ -258,7 +258,8 @@
        }
 }
 
-void rna_TextureSlot_brush_update(Main *bmain, Scene *scene, PointerRNA *ptr) {
+void rna_TextureSlot_brush_update(Main *bmain, Scene *scene, PointerRNA *ptr)
+{
        ID *id = ptr->id.data;
 
        DAG_id_tag_update(id, 0);

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to