Revision: 28636
          
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=28636
Author:   broken
Date:     2010-05-07 05:44:34 +0200 (Fri, 07 May 2010)

Log Message:
-----------
Warning cleanup

While I was looking in outliner.c, made some changes to let extra passes display
there such as environment (commented out before due to a previous limitation).

Also changed outliner object visbility/selectability/renderability toggles to 
use 
RNA buttons so you can insert keyframes with RMB menu etc.

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/colortools.c
    trunk/blender/source/blender/editors/animation/keyframing.c
    trunk/blender/source/blender/editors/interface/interface_draw.c
    trunk/blender/source/blender/editors/object/object_constraint.c
    trunk/blender/source/blender/editors/render/render_shading.c
    trunk/blender/source/blender/editors/space_outliner/outliner.c
    trunk/blender/source/blender/makesdna/DNA_scene_types.h
    trunk/blender/source/blender/makesrna/intern/rna_object.c
    trunk/blender/source/blender/python/generic/blf_api.c

Modified: trunk/blender/source/blender/blenkernel/intern/colortools.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/colortools.c 2010-05-07 
02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/blenkernel/intern/colortools.c 2010-05-07 
03:44:34 UTC (rev 28636)
@@ -939,12 +939,12 @@
 {
        int x, y, c, n, nl;
        double div, divl;
-       float *rf;
-       unsigned char *rc;
+       float *rf=NULL;
+       unsigned char *rc=NULL;
        unsigned int *bin_r, *bin_g, *bin_b, *bin_lum;
        int savedlines, saveline;
        float rgb[3], ycc[3];
-       int ycc_mode;
+       int ycc_mode=-1;
 
        if (scopes->ok == 1 ) return;
 

Modified: trunk/blender/source/blender/editors/animation/keyframing.c
===================================================================
--- trunk/blender/source/blender/editors/animation/keyframing.c 2010-05-07 
02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/editors/animation/keyframing.c 2010-05-07 
03:44:34 UTC (rev 28636)
@@ -1033,9 +1033,13 @@
        if (ELEM(NULL, sa, scene)) 
                return 0;
        
-       /* if Outliner, only allow in DataBlocks view */
-       if (so && (so->outlinevis != SO_DATABLOCKS))
-               return 0;
+       /* if Outliner, don't allow in some views */
+       if (so) {
+               if (ELEM4(so->outlinevis, SO_GROUPS, SO_LIBRARIES, 
SO_VERSE_SESSION, SO_VERSE_SESSION))
+                       return 0;
+               if (ELEM3(so->outlinevis, SO_SEQUENCE, SO_USERDEF, SO_KEYMAP))
+                       return 0;
+       }
        
        /* TODO: checks for other space types can be added here */
        

Modified: trunk/blender/source/blender/editors/interface/interface_draw.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_draw.c     
2010-05-07 02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/editors/interface/interface_draw.c     
2010-05-07 03:44:34 UTC (rev 28636)
@@ -969,7 +969,7 @@
 void vectorscope_draw_target(float centerx, float centery, float diam, float 
r, float g, float b)
 {
        float y,u,v;
-       float tangle, tampli;
+       float tangle=0.f, tampli;
        float dangle, dampli, dangle2, dampli2;
 
        rgb_to_yuv(r,g,b, &y, &u, &v);

Modified: trunk/blender/source/blender/editors/object/object_constraint.c
===================================================================
--- trunk/blender/source/blender/editors/object/object_constraint.c     
2010-05-07 02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/editors/object/object_constraint.c     
2010-05-07 03:44:34 UTC (rev 28636)
@@ -496,7 +496,7 @@
        char constraint_name[32];
        int owner = RNA_enum_get(op->ptr, "owner");
        bConstraint *con;
-       ListBase *list;
+       ListBase *list=NULL;
        
        RNA_string_get(op->ptr, "constraint", constraint_name);
        

Modified: trunk/blender/source/blender/editors/render/render_shading.c
===================================================================
--- trunk/blender/source/blender/editors/render/render_shading.c        
2010-05-07 02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/editors/render/render_shading.c        
2010-05-07 03:44:34 UTC (rev 28636)
@@ -796,7 +796,7 @@
 
 static int save_envmap(wmOperator *op, Scene *scene, EnvMap *env, char *str, 
int imtype)
 {
-       ImBuf *ibuf;
+       ImBuf *ibuf=NULL;
        int dx;
        int retval;
        

Modified: trunk/blender/source/blender/editors/space_outliner/outliner.c
===================================================================
--- trunk/blender/source/blender/editors/space_outliner/outliner.c      
2010-05-07 02:37:05 UTC (rev 28635)
+++ trunk/blender/source/blender/editors/space_outliner/outliner.c      
2010-05-07 03:44:34 UTC (rev 28636)
@@ -27,6 +27,7 @@
  * ***** END GPL LICENSE BLOCK *****
  */
 
+#include <math.h>
 #include <string.h>
 #include <stdlib.h>
 #include <stddef.h>
@@ -433,13 +434,17 @@
 static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void 
*idv, 
                                                                                
 TreeElement *parent, short type, short index);
 
+#define LOG2I(x) (int)(log(x)/log(2.0))
 
 static void outliner_add_passes(SpaceOops *soops, TreeElement *tenla, ID *id, 
SceneRenderLayer *srl)
 {
        TreeStoreElem *tselem= TREESTORE(tenla);
        TreeElement *te;
+
+       /* log stuff is to convert bitflags (powers of 2) to small integers,
+        * in order to not overflow short tselem->nr */
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_COMBINED);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_COMBINED));
        te->name= "Combined";
        te->directdata= &srl->passflag;
        
@@ -447,73 +452,72 @@
        if(tselem->flag & TSE_CLOSED)
                return;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_Z);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_Z));
        te->name= "Z";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_VECTOR);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_VECTOR));
        te->name= "Vector";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_NORMAL);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_NORMAL));
        te->name= "Normal";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_UV);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_UV));
        te->name= "UV";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_MIST);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_MIST));
        te->name= "Mist";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_INDEXOB);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_INDEXOB));
        te->name= "Index Object";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_RGBA);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_RGBA));
        te->name= "Color";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_DIFFUSE);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_DIFFUSE));
        te->name= "Diffuse";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_SPEC);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_SPEC));
        te->name= "Specular";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_SHADOW);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_SHADOW));
        te->name= "Shadow";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_AO);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_AO));
        te->name= "AO";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_REFLECT);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_REFLECT));
        te->name= "Reflection";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_REFRACT);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_REFRACT));
        te->name= "Refraction";
        te->directdata= &srl->passflag;
        
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_INDIRECT);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_INDIRECT));
        te->name= "Indirect";
        te->directdata= &srl->passflag;
 
-       /* TODO SCE_PASS_ENVIRONMENT/EMIT overflow short..
-
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_ENVIRONMENT);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_ENVIRONMENT));
        te->name= "Environment";
        te->directdata= &srl->passflag;
 
-       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
SCE_PASS_EMIT);
+       te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, 
LOG2I(SCE_PASS_EMIT));
        te->name= "Emit";
-       te->directdata= &srl->passflag;*/
+       te->directdata= &srl->passflag;
 }
 
+#undef LOG2I
 
 /* special handling of hierarchical non-lib data */
 static void outliner_add_bone(SpaceOops *soops, ListBase *lb, ID *id, Bone 
*curBone, 
@@ -4977,22 +4981,29 @@
                if(te->ys+2*OL_H >= ar->v2d.cur.ymin && te->ys <= 
ar->v2d.cur.ymax) {   
                        /* objects have toggle-able restriction flags */
                        if(tselem->type==0 && te->idcode==ID_OB) {
+                               PointerRNA ptr;
+
                                ob = (Object *)tselem->id;
+                               RNA_pointer_create((ID *)ob, &RNA_Object, ob, 
&ptr);
                                
                                uiBlockSetEmboss(block, UI_EMBOSSN);
-                               bt= uiDefIconButBitS(block, ICONTOG, 
OB_RESTRICT_VIEW, 0, ICON_RESTRICT_VIEW_OFF, 
-                                               
(int)ar->v2d.cur.xmax-OL_TOG_RESTRICT_VIEWX, (short)te->ys, 17, OL_H-1, 
&(ob->restrictflag), 0, 0, 0, 0, "Restrict/Allow visibility in the 3D View");
+                               bt= uiDefIconButR(block, ICONTOG, 0, 
ICON_RESTRICT_VIEW_OFF,
+                                                         
(int)ar->v2d.cur.xmax-OL_TOG_RESTRICT_VIEWX, (short)te->ys, 17, OL_H-1,
+                                                         &ptr, 
"restrict_view", -1, 0, 0, -1, -1, NULL);
                                uiButSetFunc(bt, restrictbutton_view_cb, scene, 
ob);
                                
-                               bt= uiDefIconButBitS(block, ICONTOG, 
OB_RESTRICT_SELECT, 0, ICON_RESTRICT_SELECT_OFF, 
-                                               
(int)ar->v2d.cur.xmax-OL_TOG_RESTRICT_SELECTX, (short)te->ys, 17, OL_H-1, 
&(ob->restrictflag), 0, 0, 0, 0, "Restrict/Allow selection in the 3D View");
+                               bt= uiDefIconButR(block, ICONTOG, 0, 
ICON_RESTRICT_SELECT_OFF,
+                                                                 
(int)ar->v2d.cur.xmax-OL_TOG_RESTRICT_SELECTX, (short)te->ys, 17, OL_H-1,
+                                                                 &ptr, 
"restrict_select", -1, 0, 0, -1, -1, NULL);
                                uiButSetFunc(bt, restrictbutton_sel_cb, scene, 
ob);
                                
-                               bt= uiDefIconButBitS(block, ICONTOG, 
OB_RESTRICT_RENDER, 0, ICON_RESTRICT_RENDER_OFF, 
-                                               
(int)ar->v2d.cur.xmax-OL_TOG_RESTRICT_RENDERX, (short)te->ys, 17, OL_H-1, 
&(ob->restrictflag), 0, 0, 0, 0, "Restrict/Allow renderability");
+                               bt= uiDefIconButR(block, ICONTOG, 0, 
ICON_RESTRICT_RENDER_OFF,

@@ Diff output truncated at 10240 characters. @@

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

Reply via email to