style cleanup: follow style guide for formatting of if/for/while loops, and else...
[blender.git] / source / blender / editors / space_outliner / outliner_tree.c
index b702bea5168e4319ba23d60758bffb66bce592bb..e83fce0cf05852192c57982b759204c39b5dcd6d 100644 (file)
@@ -115,23 +115,23 @@ static void outliner_storage_cleanup(SpaceOops *soops)
 {
        TreeStore *ts= soops->treestore;
        
-       if(ts) {
+       if (ts) {
                TreeStoreElem *tselem;
                int a, unused= 0;
                
                /* each element used once, for ID blocks with more users to have each a treestore */
-               for(a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) tselem->used= 0;
+               for (a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) tselem->used= 0;
                
                /* cleanup only after reading file or undo step, and always for
                 * RNA datablocks view in order to save memory */
-               if(soops->storeflag & SO_TREESTORE_CLEANUP) {
+               if (soops->storeflag & SO_TREESTORE_CLEANUP) {
                        
-                       for(a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) {
-                               if(tselem->id==NULL) unused++;
+                       for (a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) {
+                               if (tselem->id==NULL) unused++;
                        }
                        
-                       if(unused) {
-                               if(ts->usedelem == unused) {
+                       if (unused) {
+                               if (ts->usedelem == unused) {
                                        MEM_freeN(ts->data);
                                        ts->data= NULL;
                                        ts->usedelem= ts->totelem= 0;
@@ -140,8 +140,8 @@ static void outliner_storage_cleanup(SpaceOops *soops)
                                        TreeStoreElem *tsnewar, *tsnew;
                                        
                                        tsnew=tsnewar= MEM_mallocN((ts->usedelem-unused)*sizeof(TreeStoreElem), "new tselem");
-                                       for(a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) {
-                                               if(tselem->id) {
+                                       for (a=0, tselem= ts->data; a<ts->usedelem; a++, tselem++) {
+                                               if (tselem->id) {
                                                        *tsnew= *tselem;
                                                        tsnew++;
                                                }
@@ -163,16 +163,16 @@ static void check_persistent(SpaceOops *soops, TreeElement *te, ID *id, short ty
        int a;
        
        /* case 1; no TreeStore */
-       if(soops->treestore==NULL) {
+       if (soops->treestore==NULL) {
                soops->treestore= MEM_callocN(sizeof(TreeStore), "treestore");
        }
        ts= soops->treestore;
        
        /* check if 'te' is in treestore */
        tselem= ts->data;
-       for(a=0; a<ts->usedelem; a++, tselem++) {
-               if(tselem->id==id && tselem->used==0) {
-                       if((type==0 && tselem->type==0) ||(tselem->type==type && tselem->nr==nr)) {
+       for (a=0; a<ts->usedelem; a++, tselem++) {
+               if (tselem->id==id && tselem->used==0) {
+                       if ((type==0 && tselem->type==0) ||(tselem->type==type && tselem->nr==nr)) {
                                te->store_index= a;
                                tselem->used= 1;
                                return;
@@ -181,11 +181,11 @@ static void check_persistent(SpaceOops *soops, TreeElement *te, ID *id, short ty
        }
        
        /* add 1 element to treestore */
-       if(ts->usedelem==ts->totelem) {
+       if (ts->usedelem==ts->totelem) {
                TreeStoreElem *tsnew;
                
                tsnew= MEM_mallocN((ts->totelem+TS_CHUNK)*sizeof(TreeStoreElem), "treestore data");
-               if(ts->data) {
+               if (ts->data) {
                        memcpy(tsnew, ts->data, ts->totelem*sizeof(TreeStoreElem));
                        MEM_freeN(ts->data);
                }
@@ -196,7 +196,7 @@ static void check_persistent(SpaceOops *soops, TreeElement *te, ID *id, short ty
        tselem= ts->data+ts->usedelem;
        
        tselem->type= type;
-       if(type) tselem->nr= nr; // we're picky! :)
+       if (type) tselem->nr= nr; // we're picky! :)
        else tselem->nr= 0;
        tselem->id= id;
        tselem->used = 0;
@@ -211,13 +211,13 @@ static void check_persistent(SpaceOops *soops, TreeElement *te, ID *id, short ty
 
 void outliner_free_tree(ListBase *lb)
 {
-       while(lb->first) {
+       while (lb->first) {
                TreeElement *te= lb->first;
                
                outliner_free_tree(&te->subtree);
                BLI_remlink(lb, te);
                
-               if(te->flag & TE_FREE_NAME) MEM_freeN((void *)te->name);
+               if (te->flag & TE_FREE_NAME) MEM_freeN((void *)te->name);
                MEM_freeN(te);
        }
 }
@@ -232,10 +232,10 @@ void outliner_cleanup_tree(SpaceOops *soops)
 static TreeElement *outliner_find_tree_element(ListBase *lb, int store_index)
 {
        TreeElement *te= lb->first, *tes;
-       while(te) {
-               if(te->store_index==store_index) return te;
+       while (te) {
+               if (te->store_index==store_index) return te;
                tes= outliner_find_tree_element(&te->subtree, store_index);
-               if(tes) return tes;
+               if (tes) return tes;
                te= te->next;
        }
        return NULL;
@@ -248,18 +248,18 @@ TreeElement *outliner_find_tse(SpaceOops *soops, TreeStoreElem *tse)
        TreeStoreElem *tselem;
        int a;
        
-       if(tse->id==NULL) return NULL;
+       if (tse->id==NULL) return NULL;
        
        /* check if 'tse' is in treestore */
        tselem= ts->data;
-       for(a=0; a<ts->usedelem; a++, tselem++) {
-               if((tse->type==0 && tselem->type==0) || (tselem->type==tse->type && tselem->nr==tse->nr)) {
-                       if(tselem->id==tse->id) {
+       for (a=0; a<ts->usedelem; a++, tselem++) {
+               if ((tse->type==0 && tselem->type==0) || (tselem->type==tse->type && tselem->nr==tse->nr)) {
+                       if (tselem->id==tse->id) {
                                break;
                        }
                }
        }
-       if(tselem) 
+       if (tselem) 
                return outliner_find_tree_element(&soops->tree, a);
        
        return NULL;
@@ -271,14 +271,14 @@ TreeElement *outliner_find_id(SpaceOops *soops, ListBase *lb, ID *id)
        TreeElement *te, *tes;
        TreeStoreElem *tselem;
        
-       for(te= lb->first; te; te= te->next) {
+       for (te= lb->first; te; te= te->next) {
                tselem= TREESTORE(te);
-               if(tselem->type==0) {
-                       if(tselem->id==id) return te;
+               if (tselem->type==0) {
+                       if (tselem->id==id) return te;
                        /* only deeper on scene or object */
-                       if( te->idcode==ID_OB || te->idcode==ID_SCE || (soops->outlinevis == SO_GROUPS && te->idcode==ID_GR)) {
+                       if ( te->idcode==ID_OB || te->idcode==ID_SCE || (soops->outlinevis == SO_GROUPS && te->idcode==ID_GR)) {
                                tes= outliner_find_id(soops, &te->subtree, id);
-                               if(tes) return tes;
+                               if (tes) return tes;
                        }
                }
        }
@@ -291,9 +291,9 @@ ID *outliner_search_back(SpaceOops *soops, TreeElement *te, short idcode)
        TreeStoreElem *tselem;
        te= te->parent;
        
-       while(te) {
+       while (te) {
                tselem= TREESTORE(te);
-               if(tselem->type==0 && te->idcode==idcode) return tselem->id;
+               if (tselem->type==0 && te->idcode==idcode) return tselem->id;
                te= te->parent;
        }
        return NULL;
@@ -318,7 +318,7 @@ static void outliner_add_bone(SpaceOops *soops, ListBase *lb, ID *id, Bone *curB
        te->name= curBone->name;
        te->directdata= curBone;
        
-       for(curBone= curBone->childbase.first; curBone; curBone=curBone->next) {
+       for (curBone= curBone->childbase.first; curBone; curBone=curBone->next) {
                outliner_add_bone(soops, &te->subtree, id, curBone, te, a);
        }
 }
@@ -341,7 +341,7 @@ static void outliner_add_passes(SpaceOops *soops, TreeElement *tenla, ID *id, Sc
        
        /* save cpu cycles, but we add the first to invoke an open/close triangle */
        tselem = TREESTORE(tenla);
-       if(tselem->flag & TSE_CLOSED)
+       if (tselem->flag & TSE_CLOSED)
                return;
        
        te= outliner_add_element(soops, &tenla->subtree, id, tenla, TSE_R_PASS, LOG2I(SCE_PASS_Z));
@@ -422,14 +422,14 @@ static void outliner_add_scene_contents(SpaceOops *soops, ListBase *lb, Scene *s
        int a;
        
        tenla->name= "RenderLayers";
-       for(a=0, srl= sce->r.layers.first; srl; srl= srl->next, a++) {
+       for (a=0, srl= sce->r.layers.first; srl; srl= srl->next, a++) {
                TreeElement *tenlay= outliner_add_element(soops, &tenla->subtree, sce, te, TSE_R_LAYER, a);
                tenlay->name= srl->name;
                tenlay->directdata= &srl->passflag;
                
-               if(srl->light_override)
+               if (srl->light_override)
                        outliner_add_element(soops, &tenlay->subtree, srl->light_override, tenlay, TSE_LINKED_LAMP, 0);
-               if(srl->mat_override)
+               if (srl->mat_override)
                        outliner_add_element(soops, &tenlay->subtree, srl->mat_override, tenlay, TSE_LINKED_MAT, 0);
                
                outliner_add_passes(soops, tenlay, &sce->id, srl);
@@ -469,13 +469,13 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                if ((arm->edbo == NULL) && (ob->mode & OB_MODE_POSE)) {
                        int a= 0, const_index= 1000;    /* ensure unique id for bone constraints */
                        
-                       for(pchan= ob->pose->chanbase.first; pchan; pchan= pchan->next, a++) {
+                       for (pchan= ob->pose->chanbase.first; pchan; pchan= pchan->next, a++) {
                                ten= outliner_add_element(soops, &tenla->subtree, ob, tenla, TSE_POSE_CHANNEL, a);
                                ten->name= pchan->name;
                                ten->directdata= pchan;
                                pchan->temp= (void *)ten;
                                
-                               if(pchan->constraints.first) {
+                               if (pchan->constraints.first) {
                                        //Object *target;
                                        bConstraint *con;
                                        TreeElement *ten1;
@@ -483,12 +483,12 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                                        //char *str;
                                        
                                        tenla1->name= "Constraints";
-                                       for(con= pchan->constraints.first; con; con= con->next, const_index++) {
+                                       for (con= pchan->constraints.first; con; con= con->next, const_index++) {
                                                ten1= outliner_add_element(soops, &tenla1->subtree, ob, tenla1, TSE_CONSTRAINT, const_index);
 #if 0 /* disabled as it needs to be reworked for recoded constraints system */
                                                target= get_constraint_target(con, &str);
-                                               if(str && str[0]) ten1->name= str;
-                                               else if(target) ten1->name= target->id.name+2;
+                                               if (str && str[0]) ten1->name= str;
+                                               else if (target) ten1->name= target->id.name+2;
                                                else ten1->name= con->name;
 #endif
                                                ten1->name= con->name;
@@ -499,12 +499,12 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                        }
                        /* make hierarchy */
                        ten= tenla->subtree.first;
-                       while(ten) {
+                       while (ten) {
                                TreeElement *nten= ten->next, *par;
                                tselem= TREESTORE(ten);
-                               if(tselem->type==TSE_POSE_CHANNEL) {
+                               if (tselem->type==TSE_POSE_CHANNEL) {
                                        pchan= (bPoseChannel *)ten->directdata;
-                                       if(pchan->parent) {
+                                       if (pchan->parent) {
                                                BLI_remlink(&tenla->subtree, ten);
                                                par= (TreeElement *)pchan->parent->temp;
                                                BLI_addtail(&par->subtree, ten);
@@ -516,7 +516,7 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                }
                
                /* Pose Groups */
-               if(ob->pose->agroups.first) {
+               if (ob->pose->agroups.first) {
                        bActionGroup *agrp;
                        TreeElement *ten;
                        TreeElement *tenla= outliner_add_element(soops, &te->subtree, ob, te, TSE_POSEGRP_BASE, 0);
@@ -531,10 +531,10 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                }
        }
        
-       for(a=0; a<ob->totcol; a++) 
+       for (a=0; a<ob->totcol; a++) 
                outliner_add_element(soops, &te->subtree, ob->mat[a], te, 0, a);
        
-       if(ob->constraints.first) {
+       if (ob->constraints.first) {
                //Object *target;
                bConstraint *con;
                TreeElement *ten;
@@ -546,8 +546,8 @@ static void outliner_add_object_contents(SpaceOops *soops, TreeElement *te, Tree
                        ten= outliner_add_element(soops, &tenla->subtree, ob, tenla, TSE_CONSTRAINT, a);
 #if 0 /* disabled due to constraints system targets recode... code here needs review */
                        target= get_constraint_target(con, &str);
-                       if(str && str[0]) ten->name= str;
-                       else if(target) ten->name= target->id.name+2;
+                       if (str && str[0]) ten->name= str;
+                       else if (target) ten->name= target->id.name+2;
                        else ten->name= con->name;
 #endif
                        ten->name= con->name;
@@ -641,7 +641,7 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                                outliner_add_element(soops, &te->subtree, me, te, TSE_ANIM_DATA, 0);
                        
                        outliner_add_element(soops, &te->subtree, me->key, te, 0, 0);
-                       for(a=0; a<me->totcol; a++) 
+                       for (a=0; a<me->totcol; a++) 
                                outliner_add_element(soops, &te->subtree, me->mat[a], te, 0, a);
                        /* could do tfaces with image links, but the images are not grouped nicely.
                         * would require going over all tfaces, sort images in use. etc... */
@@ -655,7 +655,7 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (cu->adt)
                                outliner_add_element(soops, &te->subtree, cu, te, TSE_ANIM_DATA, 0);
                        
-                       for(a=0; a<cu->totcol; a++) 
+                       for (a=0; a<cu->totcol; a++) 
                                outliner_add_element(soops, &te->subtree, cu->mat[a], te, 0, a);
                }
                        break;
@@ -667,7 +667,7 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (mb->adt)
                                outliner_add_element(soops, &te->subtree, mb, te, TSE_ANIM_DATA, 0);
                        
-                       for(a=0; a<mb->totcol; a++) 
+                       for (a=0; a<mb->totcol; a++) 
                                outliner_add_element(soops, &te->subtree, mb->mat[a], te, 0, a);
                }
                        break;
@@ -679,8 +679,8 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (ma->adt)
                                outliner_add_element(soops, &te->subtree, ma, te, TSE_ANIM_DATA, 0);
                        
-                       for(a=0; a<MAX_MTEX; a++) {
-                               if(ma->mtex[a]) outliner_add_element(soops, &te->subtree, ma->mtex[a]->tex, te, 0, a);
+                       for (a=0; a<MAX_MTEX; a++) {
+                               if (ma->mtex[a]) outliner_add_element(soops, &te->subtree, ma->mtex[a]->tex, te, 0, a);
                        }
                }
                        break;
@@ -710,8 +710,8 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (la->adt)
                                outliner_add_element(soops, &te->subtree, la, te, TSE_ANIM_DATA, 0);
                        
-                       for(a=0; a<MAX_MTEX; a++) {
-                               if(la->mtex[a]) outliner_add_element(soops, &te->subtree, la->mtex[a]->tex, te, 0, a);
+                       for (a=0; a<MAX_MTEX; a++) {
+                               if (la->mtex[a]) outliner_add_element(soops, &te->subtree, la->mtex[a]->tex, te, 0, a);
                        }
                }
                        break;
@@ -719,7 +719,7 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        {
                                Speaker *spk= (Speaker *)id;
 
-                               if(spk->adt)
+                               if (spk->adt)
                                        outliner_add_element(soops, &te->subtree, spk, te, TSE_ANIM_DATA, 0);
                        }
                        break;
@@ -731,8 +731,8 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (wrld->adt)
                                outliner_add_element(soops, &te->subtree, wrld, te, TSE_ANIM_DATA, 0);
                        
-                       for(a=0; a<MAX_MTEX; a++) {
-                               if(wrld->mtex[a]) outliner_add_element(soops, &te->subtree, wrld->mtex[a]->tex, te, 0, a);
+                       for (a=0; a<MAX_MTEX; a++) {
+                               if (wrld->mtex[a]) outliner_add_element(soops, &te->subtree, wrld->mtex[a]->tex, te, 0, a);
                        }
                }
                        break;
@@ -758,7 +758,7 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        if (arm->adt)
                                outliner_add_element(soops, &te->subtree, arm, te, TSE_ANIM_DATA, 0);
                        
-                       if(arm->edbo) {
+                       if (arm->edbo) {
                                EditBone *ebone;
                                TreeElement *ten;
                                
@@ -770,10 +770,10 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                                }
                                /* make hierarchy */
                                ten= arm->edbo->first ? ((EditBone *)arm->edbo->first)->temp : NULL;
-                               while(ten) {
+                               while (ten) {
                                        TreeElement *nten= ten->next, *par;
                                        ebone= (EditBone *)ten->directdata;
-                                       if(ebone->parent) {
+                                       if (ebone->parent) {
                                                BLI_remlink(&te->subtree, ten);
                                                par= ebone->parent->temp;
                                                BLI_addtail(&par->subtree, ten);
@@ -785,10 +785,10 @@ static void outliner_add_id_contents(SpaceOops *soops, TreeElement *te, TreeStor
                        else {
                                /* do not extend Armature when we have posemode */
                                tselem= TREESTORE(te->parent);
-                               if( GS(tselem->id->name)==ID_OB && ((Object *)tselem->id)->mode & OB_MODE_POSE);
+                               if ( GS(tselem->id->name)==ID_OB && ((Object *)tselem->id)->mode & OB_MODE_POSE);
                                else {
                                        Bone *curBone;
-                                       for (curBone=arm->bonebase.first; curBone; curBone=curBone->next){
+                                       for (curBone=arm->bonebase.first; curBone; curBone=curBone->next) {
                                                outliner_add_bone(soops, &te->subtree, id, curBone, te, &a);
                                        }
                                }
@@ -807,12 +807,12 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
        ID *id= idv;
        int a = 0;
        
-       if(ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM)) {
+       if (ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM)) {
                id= ((PointerRNA*)idv)->id.data;
-               if(!id) id= ((PointerRNA*)idv)->data;
+               if (!id) id= ((PointerRNA*)idv)->data;
        }
 
-       if(id==NULL) return NULL;
+       if (id==NULL) return NULL;
 
        te= MEM_callocN(sizeof(TreeElement), "tree elem");
        /* add to the visual tree */
@@ -822,24 +822,24 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
        tselem= TREESTORE(te);  
        
        /* if we are searching for something expand to see child elements */
-       if(SEARCHING_OUTLINER(soops))
+       if (SEARCHING_OUTLINER(soops))
                tselem->flag |= TSE_CHILDSEARCH;
        
        te->parent= parent;
        te->index= index;       // for data arays
-       if(ELEM3(type, TSE_SEQUENCE, TSE_SEQ_STRIP, TSE_SEQUENCE_DUP));
-       else if(ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM));
-       else if(type==TSE_ANIM_DATA);
+       if (ELEM3(type, TSE_SEQUENCE, TSE_SEQ_STRIP, TSE_SEQUENCE_DUP));
+       else if (ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM));
+       else if (type==TSE_ANIM_DATA);
        else {
                te->name= id->name+2; // default, can be overridden by Library or non-ID data
                te->idcode= GS(id->name);
        }
        
-       if(type==0) {
+       if (type==0) {
                /* ID datablock */
                outliner_add_id_contents(soops, te, tselem, id);
        }
-       else if(type==TSE_ANIM_DATA) {
+       else if (type==TSE_ANIM_DATA) {
                IdAdtTemplate *iat = (IdAdtTemplate *)idv;
                AnimData *adt= (AnimData *)iat->adt;
                
@@ -897,12 +897,12 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                                
                                for (strip=nlt->strips.first; strip; strip=strip->next, b++) {
                                        ten= outliner_add_element(soops, &tenlt->subtree, strip->act, tenlt, TSE_NLA_ACTION, b);
-                                       if(ten) ten->directdata= strip;
+                                       if (ten) ten->directdata= strip;
                                }
                        }
                }
        }
-       else if(type==TSE_SEQUENCE) {
+       else if (type==TSE_SEQUENCE) {
                Sequence *seq= (Sequence*) idv;
                Sequence *p;
 
@@ -914,25 +914,25 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                te->idcode= seq->type;
                te->directdata= seq;
 
-               if(seq->type<7) {
+               if (seq->type<7) {
                        /*
                         * This work like the sequence.
                         * If the sequence have a name (not default name)
                         * show it, in other case put the filename.
                         */
-                       if(strcmp(seq->name, "SQ"))
+                       if (strcmp(seq->name, "SQ"))
                                te->name= seq->name;
                        else {
-                               if((seq->strip) && (seq->strip->stripdata))
+                               if ((seq->strip) && (seq->strip->stripdata))
                                        te->name= seq->strip->stripdata->name;
                                else
                                        te->name= "SQ None";
                        }
 
-                       if(seq->type==SEQ_META) {
+                       if (seq->type==SEQ_META) {
                                te->name= "Meta Strip";
                                p= seq->seqbase.first;
-                               while(p) {
+                               while (p) {
                                        outliner_add_element(soops, &te->subtree, (void*)p, te, TSE_SEQUENCE, index);
                                        p= p->next;
                                }
@@ -943,23 +943,23 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                else
                        te->name= "Effect";
        }
-       else if(type==TSE_SEQ_STRIP) {
+       else if (type==TSE_SEQ_STRIP) {
                Strip *strip= (Strip *)idv;
 
-               if(strip->dir)
+               if (strip->dir)
                        te->name= strip->dir;
                else
                        te->name= "Strip None";
                te->directdata= strip;
        }
-       else if(type==TSE_SEQUENCE_DUP) {
+       else if (type==TSE_SEQUENCE_DUP) {
                Sequence *seq= (Sequence*)idv;
 
                te->idcode= seq->type;
                te->directdata= seq;
                te->name= seq->strip->stripdata->name;
        }
-       else if(ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM)) {
+       else if (ELEM3(type, TSE_RNA_STRUCT, TSE_RNA_PROPERTY, TSE_RNA_ARRAY_ELEM)) {
                PointerRNA pptr, propptr, *ptr= (PointerRNA*)idv;
                PropertyRNA *prop, *iterprop;
                PropertyType proptype;
@@ -967,39 +967,39 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
 
                /* we do lazy build, for speed and to avoid infinite recusion */
 
-               if(ptr->data == NULL) {
+               if (ptr->data == NULL) {
                        te->name= "(empty)";
                }
-               else if(type == TSE_RNA_STRUCT) {
+               else if (type == TSE_RNA_STRUCT) {
                        /* struct */
                        te->name= RNA_struct_name_get_alloc(ptr, NULL, 0, NULL);
 
-                       if(te->name)
+                       if (te->name)
                                te->flag |= TE_FREE_NAME;
                        else
                                te->name= RNA_struct_ui_name(ptr->type);
 
                        /* If searching don't expand RNA entries */
-                       if(SEARCHING_OUTLINER(soops) && BLI_strcasecmp("RNA",te->name)==0) tselem->flag &= ~TSE_CHILDSEARCH;
+                       if (SEARCHING_OUTLINER(soops) && BLI_strcasecmp("RNA",te->name)==0) tselem->flag &= ~TSE_CHILDSEARCH;
 
                        iterprop= RNA_struct_iterator_property(ptr->type);
                        tot= RNA_property_collection_length(ptr, iterprop);
 
                        /* auto open these cases */
-                       if(!parent || (RNA_property_type(parent->directdata)) == PROP_POINTER)
-                               if(!tselem->used)
+                       if (!parent || (RNA_property_type(parent->directdata)) == PROP_POINTER)
+                               if (!tselem->used)
                                        tselem->flag &= ~TSE_CLOSED;
 
-                       if(TSELEM_OPEN(tselem,soops)) {
-                               for(a=0; a<tot; a++)
+                       if (TSELEM_OPEN(tselem,soops)) {
+                               for (a=0; a<tot; a++)
                                        outliner_add_element(soops, &te->subtree, (void*)ptr, te, TSE_RNA_PROPERTY, a);
                        }
-                       else if(tot)
+                       else if (tot)
                                te->flag |= TE_LAZY_CLOSED;
 
                        te->rnaptr= *ptr;
                }
-               else if(type == TSE_RNA_PROPERTY) {
+               else if (type == TSE_RNA_PROPERTY) {
                        /* property */
                        iterprop= RNA_struct_iterator_property(ptr->type);
                        RNA_property_collection_lookup_int(ptr, iterprop, index, &propptr);
@@ -1012,42 +1012,42 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                        te->rnaptr= *ptr;
 
                        /* If searching don't expand RNA entries */
-                       if(SEARCHING_OUTLINER(soops) && BLI_strcasecmp("RNA",te->name)==0) tselem->flag &= ~TSE_CHILDSEARCH;
+                       if (SEARCHING_OUTLINER(soops) && BLI_strcasecmp("RNA",te->name)==0) tselem->flag &= ~TSE_CHILDSEARCH;
 
-                       if(proptype == PROP_POINTER) {
+                       if (proptype == PROP_POINTER) {
                                pptr= RNA_property_pointer_get(ptr, prop);
 
-                               if(pptr.data) {
-                                       if(TSELEM_OPEN(tselem,soops))
+                               if (pptr.data) {
+                                       if (TSELEM_OPEN(tselem,soops))
                                                outliner_add_element(soops, &te->subtree, (void*)&pptr, te, TSE_RNA_STRUCT, -1);
                                        else
                                                te->flag |= TE_LAZY_CLOSED;
                                }
                        }
-                       else if(proptype == PROP_COLLECTION) {
+                       else if (proptype == PROP_COLLECTION) {
                                tot= RNA_property_collection_length(ptr, prop);
 
-                               if(TSELEM_OPEN(tselem,soops)) {
-                                       for(a=0; a<tot; a++) {
+                               if (TSELEM_OPEN(tselem,soops)) {
+                                       for (a=0; a<tot; a++) {
                                                RNA_property_collection_lookup_int(ptr, prop, a, &pptr);
                                                outliner_add_element(soops, &te->subtree, (void*)&pptr, te, TSE_RNA_STRUCT, a);
                                        }
                                }
-                               else if(tot)
+                               else if (tot)
                                        te->flag |= TE_LAZY_CLOSED;
                        }
-                       else if(ELEM3(proptype, PROP_BOOLEAN, PROP_INT, PROP_FLOAT)) {
+                       else if (ELEM3(proptype, PROP_BOOLEAN, PROP_INT, PROP_FLOAT)) {
                                tot= RNA_property_array_length(ptr, prop);
 
-                               if(TSELEM_OPEN(tselem,soops)) {
-                                       for(a=0; a<tot; a++)
+                               if (TSELEM_OPEN(tselem,soops)) {
+                                       for (a=0; a<tot; a++)
                                                outliner_add_element(soops, &te->subtree, (void*)ptr, te, TSE_RNA_ARRAY_ELEM, a);
                                }
-                               else if(tot)
+                               else if (tot)
                                        te->flag |= TE_LAZY_CLOSED;
                        }
                }
-               else if(type == TSE_RNA_ARRAY_ELEM) {
+               else if (type == TSE_RNA_ARRAY_ELEM) {
                        char c;
 
                        prop= parent->directdata;
@@ -1059,12 +1059,12 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                        c= RNA_property_array_item_char(prop, index);
 
                        te->name= MEM_callocN(sizeof(char)*20, "OutlinerRNAArrayName");
-                       if(c) sprintf((char *)te->name, "  %c", c);
+                       if (c) sprintf((char *)te->name, "  %c", c);
                        else sprintf((char *)te->name, "  %d", index+1);
                        te->flag |= TE_FREE_NAME;
                }
        }
-       else if(type == TSE_KEYMAP) {
+       else if (type == TSE_KEYMAP) {
                wmKeyMap *km= (wmKeyMap *)idv;
                wmKeyMapItem *kmi;
                char opname[OP_MAX_TYPENAME];
@@ -1072,24 +1072,24 @@ static TreeElement *outliner_add_element(SpaceOops *soops, ListBase *lb, void *i
                te->directdata= idv;
                te->name= km->idname;
                
-               if(TSELEM_OPEN(tselem,soops)) {
+               if (TSELEM_OPEN(tselem,soops)) {
                        a= 0;
                        
                        for (kmi = km->items.first; kmi; kmi = kmi->next, a++) {
                                const char *key= WM_key_event_string(kmi->type);
                                
-                               if(key[0]) {
+                               if (key[0]) {
                                        wmOperatorType *ot= NULL;
                                        
-                                       if(kmi->propvalue);
+                                       if (kmi->propvalue);
                                        else ot= WM_operatortype_find(kmi->idname, 0);
                                        
-                                       if(ot || kmi->propvalue) {
+                                       if (ot || kmi->propvalue) {
                                                TreeElement *ten= outliner_add_element(soops, &te->subtree, kmi, te, TSE_KEYMAP_ITEM, a);
                                                
                                                ten->directdata= kmi;
                                                
-                                               if(kmi->propvalue) {
+                                               if (kmi->propvalue) {
                                                        ten->name= "Modal map, not yet";
                                                }
                                                else {
@@ -1116,7 +1116,7 @@ static int need_add_seq_dup(Sequence *seq)
 {
        Sequence *p;
 
-       if((!seq->strip) || (!seq->strip->stripdata) || (!seq->strip->stripdata->name))
+       if ((!seq->strip) || (!seq->strip->stripdata) || (!seq->strip->stripdata->name))
                return(1);
 
        /*
@@ -1124,25 +1124,25 @@ static int need_add_seq_dup(Sequence *seq)
         * sequence before this, don't need it, just return.
         */
        p= seq->prev;
-       while(p) {
-               if((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
+       while (p) {
+               if ((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
                        p= p->prev;
                        continue;
                }
 
-               if(!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
+               if (!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
                        return(2);
                p= p->prev;
        }
 
        p= seq->next;
-       while(p) {
-               if((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
+       while (p) {
+               if ((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
                        p= p->next;
                        continue;
                }
 
-               if(!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
+               if (!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
                        return(0);
                p= p->next;
        }
@@ -1155,13 +1155,13 @@ static void outliner_add_seq_dup(SpaceOops *soops, Sequence *seq, TreeElement *t
        Sequence *p;
 
        p= seq;
-       while(p) {
-               if((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
+       while (p) {
+               if ((!p->strip) || (!p->strip->stripdata) || (!p->strip->stripdata->name)) {
                        p= p->next;
                        continue;
                }
 
-               if(!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
+               if (!strcmp(p->strip->stripdata->name, seq->strip->stripdata->name))
                        /* ch= */ /* UNUSED */ outliner_add_element(soops, &te->subtree, (void*)p, te, TSE_SEQUENCE, index);
                p= p->next;
        }
@@ -1181,18 +1181,18 @@ static void outliner_make_hierarchy(SpaceOops *soops, ListBase *lb)
        /* build hierarchy */
        // XXX also, set extents here...
        te= lb->first;
-       while(te) {
+       while (te) {
                ten= te->next;
                tselem= TREESTORE(te);
                
-               if(tselem->type==0 && te->idcode==ID_OB) {
+               if (tselem->type==0 && te->idcode==ID_OB) {
                        Object *ob= (Object *)tselem->id;
-                       if(ob->parent && ob->parent->id.newid) {
+                       if (ob->parent && ob->parent->id.newid) {
                                BLI_remlink(lb, te);
                                tep= (TreeElement *)ob->parent->id.newid;
                                BLI_addtail(&tep->subtree, te);
                                // set correct parent pointers
-                               for(te=tep->subtree.first; te; te= te->next) te->parent= tep;
+                               for (te=tep->subtree.first; te; te= te->next) te->parent= tep;
                        }
                }
                te= ten;
@@ -1216,15 +1216,15 @@ static int treesort_alpha(const void *v1, const void *v2)
        
        /* first put objects last (hierarchy) */
        comp= (x1->idcode==ID_OB);
-       if(x2->idcode==ID_OB) comp+=2;
+       if (x2->idcode==ID_OB) comp+=2;
        
-       if(comp==1) return 1;
-       else if(comp==2) return -1;
-       else if(comp==3) {
+       if (comp==1) return 1;
+       else if (comp==2) return -1;
+       else if (comp==3) {
                comp= strcmp(x1->name, x2->name);
                
-               if( comp>0 ) return 1;
-               else if( comp<0) return -1;
+               if ( comp>0 ) return 1;
+               else if ( comp<0) return -1;
                return 0;
        }
        return 0;
@@ -1237,20 +1237,20 @@ static int treesort_obtype_alpha(const void *v1, const void *v2)
        const tTreeSort *x1= v1, *x2= v2;
        
        /* first put objects last (hierarchy) */
-       if(x1->idcode==ID_OB && x2->idcode!=ID_OB) return 1;
-       else if(x2->idcode==ID_OB && x1->idcode!=ID_OB) return -1;
+       if (x1->idcode==ID_OB && x2->idcode!=ID_OB) return 1;
+       else if (x2->idcode==ID_OB && x1->idcode!=ID_OB) return -1;
        else {
                /* 2nd we check ob type */
-               if(x1->idcode==ID_OB && x2->idcode==ID_OB) {
-                       if( ((Object *)x1->id)->type > ((Object *)x2->id)->type) return 1;
-                       else if( ((Object *)x1->id)->type > ((Object *)x2->id)->type) return -1;
+               if (x1->idcode==ID_OB && x2->idcode==ID_OB) {
+                       if ( ((Object *)x1->id)->type > ((Object *)x2->id)->type) return 1;
+                       else if ( ((Object *)x1->id)->type > ((Object *)x2->id)->type) return -1;
                        else return 0;
                }
                else {
                        int comp= strcmp(x1->name, x2->name);
                        
-                       if( comp>0 ) return 1;
-                       else if( comp<0) return -1;
+                       if ( comp>0 ) return 1;
+                       else if ( comp<0) return -1;
                        return 0;
                }
        }
@@ -1265,38 +1265,38 @@ static void outliner_sort(SpaceOops *soops, ListBase *lb)
        int totelem=0;
        
        te= lb->last;
-       if(te==NULL) return;
+       if (te==NULL) return;
        tselem= TREESTORE(te);
        
        /* sorting rules; only object lists or deformgroups */
-       if( (tselem->type==TSE_DEFGROUP) || (tselem->type==0 && te->idcode==ID_OB)) {
+       if ( (tselem->type==TSE_DEFGROUP) || (tselem->type==0 && te->idcode==ID_OB)) {
                
                /* count first */
-               for(te= lb->first; te; te= te->next) totelem++;
+               for (te= lb->first; te; te= te->next) totelem++;
                
-               if(totelem>1) {
+               if (totelem>1) {
                        tTreeSort *tear= MEM_mallocN(totelem*sizeof(tTreeSort), "tree sort array");
                        tTreeSort *tp=tear;
                        int skip= 0;
                        
-                       for(te= lb->first; te; te= te->next, tp++) {
+                       for (te= lb->first; te; te= te->next, tp++) {
                                tselem= TREESTORE(te);
                                tp->te= te;
                                tp->name= te->name;
                                tp->idcode= te->idcode;
-                               if(tselem->type && tselem->type!=TSE_DEFGROUP) tp->idcode= 0;   // don't sort this
+                               if (tselem->type && tselem->type!=TSE_DEFGROUP) tp->idcode= 0;  // don't sort this
                                tp->id= tselem->id;
                        }
                        /* keep beginning of list */
-                       for(tp= tear, skip=0; skip<totelem; skip++, tp++)
-                               if(tp->idcode) break;
+                       for (tp= tear, skip=0; skip<totelem; skip++, tp++)
+                               if (tp->idcode) break;
                        
-                       if(skip<totelem)
+                       if (skip<totelem)
                                qsort(tear+skip, totelem-skip, sizeof(tTreeSort), treesort_alpha);
                        
                        lb->first=lb->last= NULL;
                        tp= tear;
-                       while(totelem--) {
+                       while (totelem--) {
                                BLI_addtail(lb, tp->te);
                                tp++;
                        }
@@ -1304,7 +1304,7 @@ static void outliner_sort(SpaceOops *soops, ListBase *lb)
                }
        }
        
-       for(te= lb->first; te; te= te->next) {
+       for (te= lb->first; te; te= te->next) {
                outliner_sort(soops, &te->subtree);
        }
 }
@@ -1379,7 +1379,7 @@ static int outliner_filter_tree(SpaceOops *soops, ListBase *lb)
                                outliner_free_tree(&te->subtree);
                                BLI_remlink(lb, te);
                                
-                               if(te->flag & TE_FREE_NAME) MEM_freeN((void *)te->name);
+                               if (te->flag & TE_FREE_NAME) MEM_freeN((void *)te->name);
                                MEM_freeN(te);
                        }
                }
@@ -1414,35 +1414,35 @@ void outliner_build_tree(Main *mainvar, Scene *scene, SpaceOops *soops)
        /* Are we looking for something - we want to tag parents to filter child matches
         * - NOT in datablocks view - searching all datablocks takes way too long to be useful
         * - this variable is only set once per tree build */
-       if(soops->search_string[0]!=0 && soops->outlinevis!=SO_DATABLOCKS)
+       if (soops->search_string[0]!=0 && soops->outlinevis!=SO_DATABLOCKS)
                soops->search_flags |= SO_SEARCH_RECURSIVE;
        else
                soops->search_flags &= ~SO_SEARCH_RECURSIVE;
 
-       if(soops->tree.first && (soops->storeflag & SO_TREESTORE_REDRAW))
+       if (soops->tree.first && (soops->storeflag & SO_TREESTORE_REDRAW))
                return;
 
        outliner_free_tree(&soops->tree);
        outliner_storage_cleanup(soops);
        
        /* clear ob id.new flags */
-       for(ob= mainvar->object.first; ob; ob= ob->id.next) ob->id.newid= NULL;
+       for (ob= mainvar->object.first; ob; ob= ob->id.next) ob->id.newid= NULL;
        
        /* options */
-       if(soops->outlinevis == SO_LIBRARIES) {
+       if (soops->outlinevis == SO_LIBRARIES) {
                Library *lib;
                
-               for(lib= mainvar->library.first; lib; lib= lib->id.next) {
+               for (lib= mainvar->library.first; lib; lib= lib->id.next) {
                        ten= outliner_add_element(soops, &soops->tree, lib, NULL, 0, 0);
                        lib->id.newid= (ID *)ten;
                }
                /* make hierarchy */
                ten= soops->tree.first;
-               while(ten) {
+               while (ten) {
                        TreeElement *nten= ten->next, *par;
                        tselem= TREESTORE(ten);
                        lib= (Library *)tselem->id;
-                       if(lib->parent) {
+                       if (lib->parent) {
                                BLI_remlink(&soops->tree, ten);
                                par= (TreeElement *)lib->parent->id.newid;
                                BLI_addtail(&par->subtree, ten);
@@ -1451,67 +1451,67 @@ void outliner_build_tree(Main *mainvar, Scene *scene, SpaceOops *soops)
                        ten= nten;
                }
                /* restore newid pointers */
-               for(lib= mainvar->library.first; lib; lib= lib->id.next)
+               for (lib= mainvar->library.first; lib; lib= lib->id.next)
                        lib->id.newid= NULL;
                
        }
-       else if(soops->outlinevis == SO_ALL_SCENES) {
+       else if (soops->outlinevis == SO_ALL_SCENES) {
                Scene *sce;
-               for(sce= mainvar->scene.first; sce; sce= sce->id.next) {
+               for (sce= mainvar->scene.first; sce; sce= sce->id.next) {
                        te= outliner_add_element(soops, &soops->tree, sce, NULL, 0, 0);
                        tselem= TREESTORE(te);
-                       if(sce==scene && show_opened) 
+                       if (sce==scene && show_opened) 
                                tselem->flag &= ~TSE_CLOSED;
                        
-                       for(base= sce->base.first; base; base= base->next) {
+                       for (base= sce->base.first; base; base= base->next) {
                                ten= outliner_add_element(soops, &te->subtree, base->object, te, 0, 0);
                                ten->directdata= base;
                        }
                        outliner_make_hierarchy(soops, &te->subtree);
                        /* clear id.newid, to prevent objects be inserted in wrong scenes (parent in other scene) */
-                       for(base= sce->base.first; base; base= base->next) base->object->id.newid= NULL;
+                       for (base= sce->base.first; base; base= base->next) base->object->id.newid= NULL;
                }
        }
-       else if(soops->outlinevis == SO_CUR_SCENE) {
+       else if (soops->outlinevis == SO_CUR_SCENE) {
                
                outliner_add_scene_contents(soops, &soops->tree, scene, NULL);
                
-               for(base= scene->base.first; base; base= base->next) {
+               for (base= scene->base.first; base; base= base->next) {
                        ten= outliner_add_element(soops, &soops->tree, base->object, NULL, 0, 0);
                        ten->directdata= base;
                }
                outliner_make_hierarchy(soops, &soops->tree);
        }
-       else if(soops->outlinevis == SO_VISIBLE) {
-               for(base= scene->base.first; base; base= base->next) {
-                       if(base->lay & scene->lay)
+       else if (soops->outlinevis == SO_VISIBLE) {
+               for (base= scene->base.first; base; base= base->next) {
+                       if (base->lay & scene->lay)
                                outliner_add_element(soops, &soops->tree, base->object, NULL, 0, 0);
                }
                outliner_make_hierarchy(soops, &soops->tree);
        }
-       else if(soops->outlinevis == SO_GROUPS) {
+       else if (soops->outlinevis == SO_GROUPS) {
                Group *group;
                GroupObject *go;
                
-               for(group= mainvar->group.first; group; group= group->id.next) {
-                       if(group->gobject.first) {
+               for (group= mainvar->group.first; group; group= group->id.next) {
+                       if (group->gobject.first) {
                                te= outliner_add_element(soops, &soops->tree, group, NULL, 0, 0);
                                
-                               for(go= group->gobject.first; go; go= go->next) {
+                               for (go= group->gobject.first; go; go= go->next) {
                                        ten= outliner_add_element(soops, &te->subtree, go->ob, te, 0, 0);
                                        ten->directdata= NULL; /* eh, why? */
                                }
                                outliner_make_hierarchy(soops, &te->subtree);
                                /* clear id.newid, to prevent objects be inserted in wrong scenes (parent in other scene) */
-                               for(go= group->gobject.first; go; go= go->next) go->ob->id.newid= NULL;
+                               for (go= group->gobject.first; go; go= go->next) go->ob->id.newid= NULL;
                        }
                }
        }
-       else if(soops->outlinevis == SO_SAME_TYPE) {
+       else if (soops->outlinevis == SO_SAME_TYPE) {
                Object *ob= OBACT;
-               if(ob) {
-                       for(base= scene->base.first; base; base= base->next) {
-                               if(base->object->type==ob->type) {
+               if (ob) {
+                       for (base= scene->base.first; base; base= base->next) {
+                               if (base->object->type==ob->type) {
                                        ten= outliner_add_element(soops, &soops->tree, base->object, NULL, 0, 0);
                                        ten->directdata= base;
                                }
@@ -1519,10 +1519,10 @@ void outliner_build_tree(Main *mainvar, Scene *scene, SpaceOops *soops)
                        outliner_make_hierarchy(soops, &soops->tree);
                }
        }
-       else if(soops->outlinevis == SO_SELECTED) {
-               for(base= scene->base.first; base; base= base->next) {
-                       if(base->lay & scene->lay) {
-                               if(base==BASACT || (base->flag & SELECT)) {
+       else if (soops->outlinevis == SO_SELECTED) {
+               for (base= scene->base.first; base; base= base->next) {
+                       if (base->lay & scene->lay) {
+                               if (base==BASACT || (base->flag & SELECT)) {
                                        ten= outliner_add_element(soops, &soops->tree, base->object, NULL, 0, 0);
                                        ten->directdata= base;
                                }
@@ -1530,65 +1530,65 @@ void outliner_build_tree(Main *mainvar, Scene *scene, SpaceOops *soops)
                }
                outliner_make_hierarchy(soops, &soops->tree);
        }
-       else if(soops->outlinevis==SO_SEQUENCE) {
+       else if (soops->outlinevis==SO_SEQUENCE) {
                Sequence *seq;
                Editing *ed= seq_give_editing(scene, FALSE);
                int op;
 
-               if(ed==NULL)
+               if (ed==NULL)
                        return;
 
                seq= ed->seqbasep->first;
-               if(!seq)
+               if (!seq)
                        return;
 
-               while(seq) {
+               while (seq) {
                        op= need_add_seq_dup(seq);
-                       if(op==1) {
+                       if (op==1) {
                                /* ten= */ outliner_add_element(soops, &soops->tree, (void*)seq, NULL, TSE_SEQUENCE, 0);
                        }
-                       else if(op==0) {
+                       else if (op==0) {
                                ten= outliner_add_element(soops, &soops->tree, (void*)seq, NULL, TSE_SEQUENCE_DUP, 0);
                                outliner_add_seq_dup(soops, seq, ten, 0);
                        }
                        seq= seq->next;
                }
        }
-       else if(soops->outlinevis==SO_DATABLOCKS) {
+       else if (soops->outlinevis==SO_DATABLOCKS) {
                PointerRNA mainptr;
 
                RNA_main_pointer_create(mainvar, &mainptr);
 
                ten= outliner_add_element(soops, &soops->tree, (void*)&mainptr, NULL, TSE_RNA_STRUCT, -1);
 
-               if(show_opened) {
+               if (show_opened) {
                        tselem= TREESTORE(ten);
                        tselem->flag &= ~TSE_CLOSED;
                }
        }
-       else if(soops->outlinevis==SO_USERDEF) {
+       else if (soops->outlinevis==SO_USERDEF) {
                PointerRNA userdefptr;
 
                RNA_pointer_create(NULL, &RNA_UserPreferences, &U, &userdefptr);
 
                ten= outliner_add_element(soops, &soops->tree, (void*)&userdefptr, NULL, TSE_RNA_STRUCT, -1);
 
-               if(show_opened) {
+               if (show_opened) {
                        tselem= TREESTORE(ten);
                        tselem->flag &= ~TSE_CLOSED;
                }
        }
-       else if(soops->outlinevis==SO_KEYMAP) {
+       else if (soops->outlinevis==SO_KEYMAP) {
                wmWindowManager *wm= mainvar->wm.first;
                wmKeyMap *km;
                
-               for(km= wm->defaultconf->keymaps.first; km; km= km->next) {
+               for (km= wm->defaultconf->keymaps.first; km; km= km->next) {
                        /* ten= */ outliner_add_element(soops, &soops->tree, (void*)km, NULL, TSE_KEYMAP, 0);
                }
        }
        else {
                ten= outliner_add_element(soops, &soops->tree, OBACT, NULL, 0, 0);
-               if(ten) ten->directdata= BASACT;
+               if (ten) ten->directdata= BASACT;
        }
 
        outliner_sort(soops, &soops->tree);