style cleanup: var naming for ui list
authorCampbell Barton <ideasman42@gmail.com>
Fri, 28 Dec 2012 10:32:49 +0000 (10:32 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 28 Dec 2012 10:32:49 +0000 (10:32 +0000)
source/blender/blenloader/intern/readfile.c
source/blender/blenloader/intern/writefile.c
source/blender/editors/include/UI_interface.h
source/blender/editors/interface/interface_handlers.c
source/blender/editors/interface/interface_layout.c
source/blender/editors/interface/interface_templates.c
source/blender/makesdna/DNA_screen_types.h
source/blender/makesrna/intern/rna_ui.c

index 520e4fd..458b642 100644 (file)
@@ -5832,7 +5832,7 @@ void blo_lib_link_screen_restore(Main *newmain, bScreen *curscreen, Scene *cursc
 static void direct_link_region(FileData *fd, ARegion *ar, int spacetype)
 {
        Panel *pa;
-       uiList *uilst;
+       uiList *ui_list;
 
        link_list(fd, &ar->panels);
 
@@ -5843,10 +5843,10 @@ static void direct_link_region(FileData *fd, ARegion *ar, int spacetype)
                pa->type = NULL;
        }
 
-       link_list(fd, &ar->uiLists);
+       link_list(fd, &ar->ui_lists);
 
-       for (uilst = ar->uiLists.first; uilst; uilst = uilst->next) {
-               uilst->type = NULL;
+       for (ui_list = ar->ui_lists.first; ui_list; ui_list = ui_list->next) {
+               ui_list->type = NULL;
        }
 
        ar->regiondata = newdataadr(fd, ar->regiondata);
index 1b28504..b1dce04 100644 (file)
@@ -2395,7 +2395,7 @@ static void write_screens(WriteData *wd, ListBase *scrbase)
                for (sa= sc->areabase.first; sa; sa= sa->next) {
                        SpaceLink *sl;
                        Panel *pa;
-                       uiList *uilst;
+                       uiList *ui_list;
                        ARegion *ar;
                        
                        writestruct(wd, DATA, "ScrArea", 1, sa);
@@ -2406,8 +2406,8 @@ static void write_screens(WriteData *wd, ListBase *scrbase)
                                for (pa= ar->panels.first; pa; pa= pa->next)
                                        writestruct(wd, DATA, "Panel", 1, pa);
                                
-                               for (uilst = ar->uiLists.first; uilst; uilst = uilst->next)
-                                       writestruct(wd, DATA, "uiList", 1, uilst);
+                               for (ui_list = ar->ui_lists.first; ui_list; ui_list = ui_list->next)
+                                       writestruct(wd, DATA, "uiList", 1, ui_list);
                        }
                        
                        sl= sa->spacedata.first;
index 281d80a..080367c 100644 (file)
@@ -780,7 +780,7 @@ uiLayout *uiLayoutRow(uiLayout *layout, int align);
 uiLayout *uiLayoutColumn(uiLayout *layout, int align);
 uiLayout *uiLayoutColumnFlow(uiLayout *layout, int number, int align);
 uiLayout *uiLayoutBox(uiLayout *layout);
-uiLayout *uiLayoutListBox(uiLayout *layout, struct uiList *uilst, struct PointerRNA *ptr, struct PropertyRNA *prop,
+uiLayout *uiLayoutListBox(uiLayout *layout, struct uiList *ui_list, struct PointerRNA *ptr, struct PropertyRNA *prop,
                           struct PointerRNA *actptr, struct PropertyRNA *actprop);
 uiLayout *uiLayoutAbsolute(uiLayout *layout, int align);
 uiLayout *uiLayoutSplit(uiLayout *layout, float percentage, int align);
index ee69168..8df47f0 100644 (file)
@@ -6095,9 +6095,9 @@ static int ui_handle_list_event(bContext *C, wmEvent *event, ARegion *ar)
        int value, min, max;
 
        if (but && (event->val == KM_PRESS)) {
-               uiList *uilst = but->custom_data;
+               uiList *ui_list = but->custom_data;
 
-               if (uilst) {
+               if (ui_list) {
                        if (ELEM(event->type, UPARROWKEY, DOWNARROWKEY) ||
                                ((ELEM(event->type, WHEELUPMOUSE, WHEELDOWNMOUSE) && event->alt)))
                        {
@@ -6109,12 +6109,12 @@ static int ui_handle_list_event(bContext *C, wmEvent *event, ARegion *ar)
                                else
                                        value++;
 
-                               CLAMP(value, 0, uilst->list_last_len - 1);
+                               CLAMP(value, 0, ui_list->list_last_len - 1);
 
-                               if (value < uilst->list_scroll)
-                                       uilst->list_scroll = value;
-                               else if (value >= uilst->list_scroll + uilst->list_size)
-                                       uilst->list_scroll = value - uilst->list_size + 1;
+                               if (value < ui_list->list_scroll)
+                                       ui_list->list_scroll = value;
+                               else if (value >= ui_list->list_scroll + ui_list->list_size)
+                                       ui_list->list_scroll = value - ui_list->list_size + 1;
 
                                RNA_property_int_range(&but->rnapoin, but->rnaprop, &min, &max);
                                value = CLAMPIS(value, min, max);
@@ -6127,27 +6127,27 @@ static int ui_handle_list_event(bContext *C, wmEvent *event, ARegion *ar)
                        }
                        else if (ELEM(event->type, WHEELUPMOUSE, WHEELDOWNMOUSE) && event->shift) {
                                /* silly replacement for proper grip */
-                               if (uilst->list_grip_size == 0)
-                                       uilst->list_grip_size = uilst->list_size;
+                               if (ui_list->list_grip_size == 0)
+                                       ui_list->list_grip_size = ui_list->list_size;
 
                                if (event->type == WHEELUPMOUSE)
-                                       uilst->list_grip_size--;
+                                       ui_list->list_grip_size--;
                                else
-                                       uilst->list_grip_size++;
+                                       ui_list->list_grip_size++;
 
-                               uilst->list_grip_size = MAX2(uilst->list_grip_size, 1);
+                               ui_list->list_grip_size = MAX2(ui_list->list_grip_size, 1);
 
                                ED_region_tag_redraw(ar);
 
                                retval = WM_UI_HANDLER_BREAK;
                        }
                        else if (ELEM(event->type, WHEELUPMOUSE, WHEELDOWNMOUSE)) {
-                               if (uilst->list_last_len > uilst->list_size) {
+                               if (ui_list->list_last_len > ui_list->list_size) {
                                        /* list template will clamp */
                                        if (event->type == WHEELUPMOUSE)
-                                               uilst->list_scroll--;
+                                               ui_list->list_scroll--;
                                        else
-                                               uilst->list_scroll++;
+                                               ui_list->list_scroll++;
 
                                        ED_region_tag_redraw(ar);
 
index 19b4442..f4af1f0 100644 (file)
@@ -2308,13 +2308,13 @@ uiLayout *uiLayoutBox(uiLayout *layout)
        return (uiLayout *)ui_layout_box(layout, ROUNDBOX);
 }
 
-uiLayout *uiLayoutListBox(uiLayout *layout, uiList *uilist, PointerRNA *ptr, PropertyRNA *prop, PointerRNA *actptr,
+uiLayout *uiLayoutListBox(uiLayout *layout, uiList *ui_list, PointerRNA *ptr, PropertyRNA *prop, PointerRNA *actptr,
                           PropertyRNA *actprop)
 {
        uiLayoutItemBx *box = ui_layout_box(layout, LISTBOX);
        uiBut *but = box->roundbox;
 
-       but->custom_data = uilist;
+       but->custom_data = ui_list;
 
        but->rnasearchpoin = *ptr;
        but->rnasearchprop = prop;
index 34f325a..a190f3d 100644 (file)
@@ -2328,7 +2328,7 @@ void uiTemplateGameStates(uiLayout *layout, PointerRNA *ptr, const char *propnam
 
 
 /************************* List Template **************************/
-static void uilist_draw_item_default(struct uiList *uilst, struct bContext *UNUSED(C), struct uiLayout *layout,
+static void uilist_draw_item_default(struct uiList *ui_list, struct bContext *UNUSED(C), struct uiLayout *layout,
                                      struct PointerRNA *UNUSED(dataptr), struct PointerRNA *itemptr, int icon,
                                      struct PointerRNA *UNUSED(active_dataptr), const char *UNUSED(active_propname),
                                      int UNUSED(index))
@@ -2340,7 +2340,7 @@ static void uilist_draw_item_default(struct uiList *uilst, struct bContext *UNUS
        name = (namebuf) ? namebuf : "";
 
        /* Simplest one! */
-       switch (uilst->layout_type) {
+       switch (ui_list->layout_type) {
        case UILST_LAYOUT_GRID:
                uiItemL(layout, "", icon);
                break;
@@ -2361,8 +2361,8 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                     PointerRNA *dataptr, const char *propname, PointerRNA *active_dataptr,
                     const char *active_propname, int rows, int maxrows, int layout_type)
 {
-       uiListType *ult;
-       uiList *uilst = NULL;
+       uiListType *ui_list_type;
+       uiList *ui_list = NULL;
        ARegion *ar;
        uiListDrawItemFunc draw_item;
 
@@ -2373,7 +2373,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
        uiBlock *block, *subblock;
        uiBut *but;
 
-       char uilst_id[UI_MAX_NAME_STR];
+       char ui_list_id[UI_MAX_NAME_STR];
        char numstr[32];
        int rnaicon = ICON_NONE, icon = ICON_NONE;
        int i = 0, activei = 0;
@@ -2428,31 +2428,31 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
        activei = RNA_property_int_get(active_dataptr, activeprop);
 
        /* Find the uiList type. */
-       ult = WM_uilisttype_find(listtype_name, FALSE);
+       ui_list_type = WM_uilisttype_find(listtype_name, FALSE);
 
-       if (ult == NULL) {
+       if (ui_list_type == NULL) {
                RNA_warning("List type %s not found", listtype_name);
                return;
        }
 
-       draw_item = ult->draw_item ? ult->draw_item : uilist_draw_item_default;
+       draw_item = ui_list_type->draw_item ? ui_list_type->draw_item : uilist_draw_item_default;
 
        /* Find or add the uiList to the current Region. */
        /* We tag the list id with the list type... */
-       BLI_snprintf(uilst_id, sizeof(uilst_id), "%s_%s", ult->idname, list_id ? list_id : "");
+       BLI_snprintf(ui_list_id, sizeof(ui_list_id), "%s_%s", ui_list_type->idname, list_id ? list_id : "");
 
        ar = CTX_wm_region(C);
-       uilst = BLI_findstring(&ar->uiLists, uilst_id, offsetof(uiList, list_id));
+       ui_list = BLI_findstring(&ar->ui_lists, ui_list_id, offsetof(uiList, list_id));
 
-       if (!uilst) {
-               uilst = MEM_callocN(sizeof(uiList), __func__);
-               BLI_strncpy(uilst->list_id, uilst_id, sizeof(uilst->list_id));
-               BLI_addtail(&ar->uiLists, uilst);
+       if (!ui_list) {
+               ui_list = MEM_callocN(sizeof(uiList), __func__);
+               BLI_strncpy(ui_list->list_id, ui_list_id, sizeof(ui_list->list_id));
+               BLI_addtail(&ar->ui_lists, ui_list);
        }
 
        /* Because we can't actually pass type across save&load... */
-       uilst->type = ult;
-       uilst->layout_type = layout_type;
+       ui_list->type = ui_list_type;
+       ui_list->layout_type = layout_type;
 
        switch (layout_type) {
        case UILST_LAYOUT_DEFAULT:
@@ -2461,11 +2461,11 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                        rows = 5;
                if (maxrows == 0)
                        maxrows = 5;
-               if (uilst->list_grip_size != 0)
-                       rows = uilst->list_grip_size;
+               if (ui_list->list_grip_size != 0)
+                       rows = ui_list->list_grip_size;
 
                /* layout */
-               box = uiLayoutListBox(layout, uilst, dataptr, prop, active_dataptr, activeprop);
+               box = uiLayoutListBox(layout, ui_list, dataptr, prop, active_dataptr, activeprop);
                row = uiLayoutRow(box, FALSE);
                col = uiLayoutColumn(row, TRUE);
 
@@ -2477,21 +2477,21 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                items = CLAMPIS(len, rows, MAX2(rows, maxrows));
 
                /* if list length changes and active is out of view, scroll to it */
-               if ((uilst->list_last_len != len) &&
-                   (activei < uilst->list_scroll || activei >= uilst->list_scroll + items)) {
-                       uilst->list_scroll = activei;
+               if ((ui_list->list_last_len != len) &&
+                   (activei < ui_list->list_scroll || activei >= ui_list->list_scroll + items)) {
+                       ui_list->list_scroll = activei;
                }
 
-               uilst->list_scroll = MIN2(uilst->list_scroll, len - items);
-               uilst->list_scroll = MAX2(uilst->list_scroll, 0);
-               uilst->list_size = items;
-               uilst->list_last_len = len;
+               ui_list->list_scroll = min_ii(ui_list->list_scroll, len - items);
+               ui_list->list_scroll = max_ii(ui_list->list_scroll, 0);
+               ui_list->list_size = items;
+               ui_list->list_last_len = len;
 
                if (dataptr->data && prop) {
                        /* create list items */
                        RNA_PROP_BEGIN (dataptr, itemptr, prop)
                        {
-                               if (i >= uilst->list_scroll && i < uilst->list_scroll + items) {
+                               if (i >= ui_list->list_scroll && i < ui_list->list_scroll + items) {
                                        subblock = uiLayoutGetBlock(col);
                                        overlap = uiLayoutOverlap(col);
 
@@ -2507,7 +2507,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                                        icon = UI_rnaptr_icon_get(C, &itemptr, rnaicon, FALSE);
                                        if (icon == ICON_DOT)
                                                icon = ICON_NONE;
-                                       draw_item(uilst, C, sub, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
+                                       draw_item(ui_list, C, sub, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
                                }
                                i++;
                        }
@@ -2515,8 +2515,8 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                }
 
                /* add dummy buttons to fill space */
-               while (i < uilst->list_scroll + items) {
-                       if (i >= uilst->list_scroll)
+               while (i < ui_list->list_scroll + items) {
+                       if (i >= ui_list->list_scroll)
                                uiItemL(col, "", ICON_NONE);
                        i++;
                }
@@ -2524,7 +2524,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                /* add scrollbar */
                if (len > items) {
                        col = uiLayoutColumn(row, FALSE);
-                       uiDefButI(block, SCROLL, 0, "", 0, 0, UI_UNIT_X * 0.75, UI_UNIT_Y * items, &uilst->list_scroll,
+                       uiDefButI(block, SCROLL, 0, "", 0, 0, UI_UNIT_X * 0.75, UI_UNIT_Y * items, &ui_list->list_scroll,
                                  0, len - items, items, 0, "");
                }
                break;
@@ -2541,7 +2541,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                                        icon = UI_rnaptr_icon_get(C, &itemptr, rnaicon, FALSE);
                                        if (icon == ICON_DOT)
                                                icon = ICON_NONE;
-                                       draw_item(uilst, C, row, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
+                                       draw_item(ui_list, C, row, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
                                }
 
                                i++;
@@ -2561,7 +2561,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                        uiButSetFlag(but, UI_BUT_DISABLED);
                break;
        case UILST_LAYOUT_GRID:
-               box = uiLayoutListBox(layout, uilst, dataptr, prop, active_dataptr, activeprop);
+               box = uiLayoutListBox(layout, ui_list, dataptr, prop, active_dataptr, activeprop);
                col = uiLayoutColumn(box, TRUE);
                row = uiLayoutRow(col, FALSE);
 
@@ -2586,7 +2586,7 @@ void uiTemplateList(uiLayout *layout, bContext *C, const char *listtype_name, co
                                sub = uiLayoutRow(overlap, FALSE);
 
                                icon = UI_rnaptr_icon_get(C, &itemptr, rnaicon, FALSE);
-                               draw_item(uilst, C, sub, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
+                               draw_item(ui_list, C, sub, dataptr, &itemptr, icon, active_dataptr, active_propname, i);
 
                                i++;
                        }
index 5d8176c..d6100dc 100644 (file)
@@ -181,7 +181,7 @@ typedef struct ARegion {
        
        ListBase uiblocks;                      /* uiBlock */
        ListBase panels;                        /* Panel */
-       ListBase uiLists;                       /* uiList */
+       ListBase ui_lists;                      /* uiList */
        ListBase handlers;                      /* wmEventHandler */
        
        struct wmTimer *regiontimer; /* blend in/out */
index e6e953c..14789a4 100644 (file)
@@ -263,19 +263,19 @@ static StructRNA *rna_Panel_refine(PointerRNA *ptr)
 }
 
 /* UIList */
-static void uilist_draw_item(uiList *uilst, bContext *C, uiLayout *layout, PointerRNA *dataptr, PointerRNA *itemptr,
+static void uilist_draw_item(uiList *ui_list, bContext *C, uiLayout *layout, PointerRNA *dataptr, PointerRNA *itemptr,
                              int icon, PointerRNA *active_dataptr, const char *active_propname, int index)
 {
        extern FunctionRNA rna_UIList_draw_item_func;
 
-       PointerRNA ultr;
+       PointerRNA ul_ptr;
        ParameterList list;
        FunctionRNA *func;
 
-       RNA_pointer_create(&CTX_wm_screen(C)->id, uilst->type->ext.srna, uilst, &ultr);
-       func = &rna_UIList_draw_item_func; /* RNA_struct_find_function(&ultr, "draw_item"); */
+       RNA_pointer_create(&CTX_wm_screen(C)->id, ui_list->type->ext.srna, ui_list, &ul_ptr);
+       func = &rna_UIList_draw_item_func; /* RNA_struct_find_function(&ul_ptr, "draw_item"); */
 
-       RNA_parameter_list_create(&list, &ultr, func);
+       RNA_parameter_list_create(&list, &ul_ptr, func);
        RNA_parameter_set_lookup(&list, "context", &C);
        RNA_parameter_set_lookup(&list, "layout", &layout);
        RNA_parameter_set_lookup(&list, "data", dataptr);
@@ -284,7 +284,7 @@ static void uilist_draw_item(uiList *uilst, bContext *C, uiLayout *layout, Point
        RNA_parameter_set_lookup(&list, "active_data", active_dataptr);
        RNA_parameter_set_lookup(&list, "active_property", &active_propname);
        RNA_parameter_set_lookup(&list, "index", &index);
-       uilst->type->ext.call((bContext *)C, &ultr, func, &list);
+       ui_list->type->ext.call((bContext *)C, &ul_ptr, func, &list);
 
        RNA_parameter_list_free(&list);
 }
@@ -311,16 +311,16 @@ static StructRNA *rna_UIList_register(Main *bmain, ReportList *reports, void *da
 {
        uiListType *ult, dummyult = {NULL};
        uiList dummyuilist = {NULL};
-       PointerRNA dummyultr;
+       PointerRNA dummyul_ptr;
        int have_function[1];
        size_t over_alloc = 0; /* warning, if this becomes a bess, we better do another alloc */
 
        /* setup dummy menu & menu type to store static properties in */
        dummyuilist.type = &dummyult;
-       RNA_pointer_create(NULL, &RNA_UIList, &dummyuilist, &dummyultr);
+       RNA_pointer_create(NULL, &RNA_UIList, &dummyuilist, &dummyul_ptr);
 
        /* validate the python class */
-       if (validate(&dummyultr, data, have_function) != 0)
+       if (validate(&dummyul_ptr, data, have_function) != 0)
                return NULL;
 
        if (strlen(identifier) >= sizeof(dummyult.idname)) {
@@ -357,8 +357,8 @@ static StructRNA *rna_UIList_register(Main *bmain, ReportList *reports, void *da
 
 static StructRNA *rna_UIList_refine(PointerRNA *ptr)
 {
-       uiList *uilst = (uiList *)ptr->data;
-       return (uilst->type && uilst->type->ext.srna) ? uilst->type->ext.srna : &RNA_UIList;
+       uiList *ui_list = (uiList *)ptr->data;
+       return (ui_list->type && ui_list->type->ext.srna) ? ui_list->type->ext.srna : &RNA_UIList;
 }
 
 /* Header */