no functional changes: SETLOOPER macro assumed a scene was defined called 'sce' used...
authorCampbell Barton <ideasman42@gmail.com>
Fri, 17 Dec 2010 15:37:59 +0000 (15:37 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 17 Dec 2010 15:37:59 +0000 (15:37 +0000)
release/scripts/presets/keyconfig/maya.py
source/blender/blenkernel/BKE_scene.h
source/blender/blenkernel/intern/collision.c
source/blender/blenkernel/intern/depsgraph.c
source/blender/blenkernel/intern/pointcache.c
source/blender/blenkernel/intern/scene.c
source/blender/editors/space_view3d/view3d_draw.c
source/blender/gpu/intern/gpu_material.c
source/blender/makesrna/intern/rna_scene.c
source/blender/render/intern/source/convertblender.c
source/gameengine/Converter/BL_BlenderDataConversion.cpp

index 29533fd7a41ac6716a44d4fb9df5f5e28e1e2962..120597cdd2afb217a38991894c7b4d19cbaabea3 100644 (file)
@@ -173,7 +173,7 @@ kmi = km.items.new('wm.context_set_enum', 'COMMA', 'PRESS', ctrl=True)
 kmi.properties.data_path = 'space_data.pivot_point'
 kmi.properties.value = 'MEDIAN_POINT'
 kmi = km.items.new('wm.context_toggle', 'COMMA', 'PRESS', alt=True)
-kmi.properties.data_path = 'space_data.use_pivot_point_align'
+kmi.properties.data_path = 'space_data.use_pivot_point'
 kmi = km.items.new('wm.context_toggle', 'Q', 'PRESS')
 kmi.properties.data_path = 'space_data.show_manipulator'
 kmi = km.items.new('wm.context_set_enum', 'PERIOD', 'PRESS')
index 4b3d3723ab571565502b72e4eec215a08be6078f..f75e4020cb9acaa20fd542517a6b54946fcd98f1 100644 (file)
@@ -51,8 +51,8 @@ struct Text;
 #define SCE_COPY_LINK_DATA     2
 #define SCE_COPY_FULL          3
 
-#define SETLOOPER(s, b) sce= s, b= _setlooper_base_step(&sce, NULL); b; b= _setlooper_base_step(&sce, b)
-struct Base *_setlooper_base_step(struct Scene **sce, struct Base *base);
+#define SETLOOPER(_sce_basis, _sce_iter, _base) _sce_iter= _sce_basis, _base= _setlooper_base_step(&_sce_iter, NULL); _base; _base= _setlooper_base_step(&_sce_iter, _base)
+struct Base *_setlooper_base_step(struct Scene **sce_iter, struct Base *base);
 
 void free_avicodecdata(struct AviCodecData *acd);
 void free_qtcodecdata(struct QuicktimeCodecData *acd);
index d39e550192b13b3eb40f072b848f334c89356a81..b2d587151b1df67d771da1406172eb0a3e9949e1 100644 (file)
@@ -1358,9 +1358,9 @@ Object **get_collisionobjects(Scene *scene, Object *self, Group *group, unsigned
                        add_collision_object(&objs, &numobj, &maxobj, go->ob, self, 0);
        }
        else {
-               Scene *sce; /* for SETLOOPER macro */
+               Scene *sce_iter;
                /* add objects in same layer in scene */
-               for(SETLOOPER(scene, base)) {
+               for(SETLOOPER(scene, sce_iter, base)) {
                        if(base->lay & self->lay)
                                add_collision_object(&objs, &numobj, &maxobj, base->object, self, 0);
 
@@ -1417,11 +1417,11 @@ ListBase *get_collider_cache(Scene *scene, Object *self, Group *group)
                        add_collider_cache_object(&objs, go->ob, self, 0);
        }
        else {
-               Scene *sce; /* for SETLOOPER macro */
+               Scene *sce_iter;
                Base *base;
 
                /* add objects in same layer in scene */
-               for(SETLOOPER(scene, base)) {
+               for(SETLOOPER(scene, sce_iter, base)) {
                        if(!self || (base->lay & self->lay))
                                add_collider_cache_object(&objs, base->object, self, 0);
 
index 2ac3946cb984cd0304e44e6b16bc045d9b419aa4..1904e63c2bab1011f93f27a2a164b5019845a7f0 100644 (file)
@@ -2139,10 +2139,10 @@ void DAG_scene_update_flags(Main *bmain, Scene *scene, unsigned int lay, const s
        Object *ob;
        Group *group;
        GroupObject *go;
-       Scene *sce;
+       Scene *sce_iter;
 
        /* set ob flags where animated systems are */
-       for(SETLOOPER(scene, base)) {
+       for(SETLOOPER(scene, sce_iter, base)) {
                ob= base->object;
 
                if(do_time) {
@@ -2167,12 +2167,12 @@ void DAG_scene_update_flags(Main *bmain, Scene *scene, unsigned int lay, const s
                }
        }
 
-       for(sce= scene; sce; sce= sce->set)
-               DAG_scene_flush_update(bmain, sce, lay, 1);
+       for(sce_iter= scene; sce_iter; sce_iter= sce_iter->set)
+               DAG_scene_flush_update(bmain, sce_iter, lay, 1);
        
        if(do_time) {
                /* test: set time flag, to disable baked systems to update */
-               for(SETLOOPER(scene, base)) {
+               for(SETLOOPER(scene, sce_iter, base)) {
                        ob= base->object;
                        if(ob->recalc)
                                ob->recalc |= OB_RECALC_TIME;
@@ -2240,7 +2240,7 @@ void DAG_ids_flush_update(Main *bmain, int time)
 
 void DAG_on_load_update(Main *bmain, const short do_time)
 {
-       Scene *scene, *sce;
+       Scene *scene;
        Base *base;
        Object *ob;
        Group *group;
@@ -2251,15 +2251,16 @@ void DAG_on_load_update(Main *bmain, const short do_time)
        dag_current_scene_layers(bmain, &scene, &lay);
 
        if(scene && scene->theDag) {
+               Scene *sce_iter;
                /* derivedmeshes and displists are not saved to file so need to be
                   remade, tag them so they get remade in the scene update loop,
                   note armature poses or object matrices are preserved and do not
                   require updates, so we skip those */
                dag_scene_flush_layers(scene, lay);
 
-               for(SETLOOPER(scene, base)) {
+               for(SETLOOPER(scene, sce_iter, base)) {
                        ob= base->object;
-                       node= (sce->theDag)? dag_get_node(sce->theDag, ob): NULL;
+                       node= (sce_iter->theDag)? dag_get_node(sce_iter->theDag, ob): NULL;
                        oblay= (node)? node->lay: ob->lay;
 
                        if(oblay & lay) {
index c43f53cc4af2bb33b84472aab0f0efb7dbc9ba75..d131a3c3592fa66f0d38cbe82ba1c9d02c60c1fe 100644 (file)
@@ -2464,7 +2464,7 @@ void BKE_ptcache_make_cache(PTCacheBaker* baker)
 {
        Main *bmain = baker->main;
        Scene *scene = baker->scene;
-       Scene *sce; /* SETLOOPER macro only */
+       Scene *sce_iter; /* SETLOOPER macro only */
        Base *base;
        ListBase pidlist;
        PTCacheID *pid = baker->pid;
@@ -2535,7 +2535,7 @@ void BKE_ptcache_make_cache(PTCacheBaker* baker)
                        cache->flag &= ~PTCACHE_BAKED;
                }
        }
-       else for(SETLOOPER(scene, base)) {
+       else for(SETLOOPER(scene, sce_iter, base)) {
                /* cache/bake everything in the scene */
                BKE_ptcache_ids_from_object(&pidlist, base->object, scene, MAX_DUPLI_RECUR);
 
@@ -2624,7 +2624,7 @@ void BKE_ptcache_make_cache(PTCacheBaker* baker)
                                BKE_ptcache_write_cache(pid, 0);
                }
        }
-       else for(SETLOOPER(scene, base)) {
+       else for(SETLOOPER(scene, sce_iter, base)) {
                BKE_ptcache_ids_from_object(&pidlist, base->object, scene, MAX_DUPLI_RECUR);
 
                for(pid=pidlist.first; pid; pid=pid->next) {
index 58b7f95f9c8cfca7ab23c0437957cd103ab44773..7a8c4efb51a10d06fb3466cadeb654f8e752cdb2 100644 (file)
@@ -1077,20 +1077,20 @@ float get_render_aosss_error(RenderData *r, float error)
 }
 
 /* helper function for the SETLOOPER macro */
-Base *_setlooper_base_step(Scene **sce, Base *base)
+Base *_setlooper_base_step(Scene **sce_iter, Base *base)
 {
     if(base && base->next) {
         /* common case, step to the next */
         return base->next;
     }
-    else if(base==NULL && (*sce)->base.first) {
+       else if(base==NULL && (*sce_iter)->base.first) {
         /* first time looping, return the scenes first base */
-        return (Base *)(*sce)->base.first;
+               return (Base *)(*sce_iter)->base.first;
     }
     else {
         /* reached the end, get the next base in the set */
-        while((*sce= (*sce)->set)) {
-            base= (Base *)(*sce)->base.first;
+               while((*sce_iter= (*sce_iter)->set)) {
+                       base= (Base *)(*sce_iter)->base.first;
             if(base) {
                 return base;
             }
index 80993b45f72a58bc2ec0dc4453db7a2ba2a3df0c..6bf87e1964b366901f48824c54d8b95ab9dede08 100644 (file)
@@ -1726,7 +1726,6 @@ void draw_depth(Scene *scene, ARegion *ar, View3D *v3d, int (* func)(void *))
 {
        RegionView3D *rv3d= ar->regiondata;
        Base *base;
-       Scene *sce;
        short zbuf= v3d->zbuf;
        short flag= v3d->flag;
        float glalphaclip= U.glalphaclip;
@@ -1759,7 +1758,8 @@ void draw_depth(Scene *scene, ARegion *ar, View3D *v3d, int (* func)(void *))
        
        /* draw set first */
        if(scene->set) {
-               for(SETLOOPER(scene->set, base)) {
+               Scene *sce_iter;
+               for(SETLOOPER(scene->set, sce_iter, base)) {
                        if(v3d->lay & base->lay) {
                                if (func == NULL || func(base)) {
                                        draw_object(scene, ar, v3d, base, 0);
@@ -1881,14 +1881,14 @@ static void gpu_update_lamps_shadows(Scene *scene, View3D *v3d)
 {
        ListBase shadows;
        View3DShadow *shadow;
-       Scene *sce;
+       Scene *sce_iter;
        Base *base;
        Object *ob;
        
        shadows.first= shadows.last= NULL;
        
        /* update lamp transform and gather shadow lamps */
-       for(SETLOOPER(scene, base)) {
+       for(SETLOOPER(scene, sce_iter, base)) {
                ob= base->object;
                
                if(ob->type == OB_LAMP)
@@ -2036,7 +2036,6 @@ static void view3d_main_area_setup_view(Scene *scene, View3D *v3d, ARegion *ar,
 
 void ED_view3d_draw_offscreen(Scene *scene, View3D *v3d, ARegion *ar, int winx, int winy, float viewmat[][4], float winmat[][4])
 {
-       Scene *sce;
        Base *base;
        float backcol[3];
        int bwinx, bwiny;
@@ -2091,7 +2090,8 @@ void ED_view3d_draw_offscreen(Scene *scene, View3D *v3d, ARegion *ar, int winx,
 
        /* draw set first */
        if(scene->set) {
-               for(SETLOOPER(scene->set, base)) {
+               Scene *sce_iter;
+               for(SETLOOPER(scene->set, sce_iter, base)) {
                        if(v3d->lay & base->lay) {
                                UI_ThemeColorBlend(TH_WIRE, TH_BACK, 0.6f);
                                draw_object(scene, ar, v3d, base, DRAW_CONSTCOLOR|DRAW_SCENESET);
@@ -2295,7 +2295,6 @@ void view3d_main_area_draw(const bContext *C, ARegion *ar)
        Scene *scene= CTX_data_scene(C);
        View3D *v3d = CTX_wm_view3d(C);
        RegionView3D *rv3d= CTX_wm_region_view3d(C);
-       Scene *sce;
        Base *base;
        Object *ob;
        float backcol[3];
@@ -2389,7 +2388,8 @@ void view3d_main_area_draw(const bContext *C, ARegion *ar)
 
        /* draw set first */
        if(scene->set) {
-               for(SETLOOPER(scene->set, base)) {
+               Scene *sce_iter;
+               for(SETLOOPER(scene->set, sce_iter, base)) {
                        
                        if(v3d->lay & base->lay) {
                                
index 592cf809a4f9a79a309badab91f8a39d4d64bdb2..b9da22b36a7e5a7d1f03ef76fd24eaed2b8a0cbe 100644 (file)
@@ -756,10 +756,10 @@ static void material_lights(GPUShadeInput *shi, GPUShadeResult *shr)
 {
        Base *base;
        Object *ob;
-       Scene *sce;
+       Scene *sce_iter;
        GPULamp *lamp;
        
-       for(SETLOOPER(shi->gpumat->scene, base)) {
+       for(SETLOOPER(shi->gpumat->scene, sce_iter, base)) {
                ob= base->object;
 
                if(ob->type==OB_LAMP) {
index 02b05cb7073c4a67a096a722ebd2eaa30665fcf8..48de2930bd038e9a1ffc713d3ddf774aa89d49f9 100644 (file)
@@ -874,10 +874,10 @@ static void object_simplify_update(Object *ob)
 
 static void rna_Scene_use_simplify_update(Main *bmain, Scene *scene, PointerRNA *ptr)
 {
-       Scene *sce;
+       Scene *sce_iter;
        Base *base;
 
-       for(SETLOOPER(scene, base))
+       for(SETLOOPER(scene, sce_iter, base))
                object_simplify_update(base->object);
        
        DAG_ids_flush_update(bmain, 0);
index 53f418966e29d412045b67b82269c1c1ca676b5f..b851728653a8608a6dd4f069b3b0fc72d1589441 100644 (file)
@@ -4679,7 +4679,7 @@ static void database_init_objects(Render *re, unsigned int renderlay, int nolamp
        Object *ob;
        Group *group;
        ObjectInstanceRen *obi;
-       Scene *sce;
+       Scene *sce_iter;
        float mat[4][4];
        int lay, vectorlay, redoimat= 0;
 
@@ -4688,7 +4688,7 @@ static void database_init_objects(Render *re, unsigned int renderlay, int nolamp
         * NULL is just for init */
        set_dupli_tex_mat(NULL, NULL, NULL);
 
-       for(SETLOOPER(re->scene, base)) {
+       for(SETLOOPER(re->scene, sce_iter, base)) {
                ob= base->object;
                /* imat objects has to be done here, since displace can have texture using Object map-input */
                mul_m4_m4m4(mat, ob->obmat, re->viewmat);
@@ -4698,7 +4698,7 @@ static void database_init_objects(Render *re, unsigned int renderlay, int nolamp
                ob->transflag &= ~OB_RENDER_DUPLI;
        }
 
-       for(SETLOOPER(re->scene, base)) {
+       for(SETLOOPER(re->scene, sce_iter, base)) {
                ob= base->object;
 
                /* in the prev/next pass for making speed vectors, avoid creating
@@ -4836,7 +4836,7 @@ static void database_init_objects(Render *re, unsigned int renderlay, int nolamp
 
        /* imat objects has to be done again, since groups can mess it up */
        if(redoimat) {
-               for(SETLOOPER(re->scene, base)) {
+               for(SETLOOPER(re->scene, sce_iter, base)) {
                        ob= base->object;
                        mul_m4_m4m4(mat, ob->obmat, re->viewmat);
                        invert_m4_m4(ob->imat, mat);
index 79afedb5f43e0d899f8116852f2188b56a23ab20..47e31585148397000adf124d5819450c77eb00af 100644 (file)
@@ -227,10 +227,10 @@ static unsigned int KX_Mcol2uint_new(MCol col)
 static void SetDefaultFaceType(Scene* scene)
 {
        default_face_mode = TF_DYNAMIC;
-       Scene *sce;
+       Scene *sce_iter;
        Base *base;
 
-       for(SETLOOPER(scene,base))
+       for(SETLOOPER(scene, sce_iter, base))
        {
                if (base->object->type == OB_LAMP)
                {
@@ -1931,7 +1931,7 @@ void BL_ConvertBlenderObjects(struct Main* maggie,
 
        Scene *blenderscene = kxscene->GetBlenderScene();
        // for SETLOOPER
-       Scene *sce;
+       Scene *sce_iter;
        Base *base;
 
        // Get the frame settings of the canvas.
@@ -2012,7 +2012,7 @@ void BL_ConvertBlenderObjects(struct Main* maggie,
        // Beware of name conflict in linked data, it will not crash but will create confusion
        // in Python scripting and in certain actuators (replace mesh). Linked scene *should* have
        // no conflicting name for Object, Object data and Action.
-       for (SETLOOPER(blenderscene, base))
+       for (SETLOOPER(blenderscene, sce_iter, base))
        {
                Object* blenderobject = base->object;
                allblobj.insert(blenderobject);