diff options
| author | Dalai Felinto <dalai@blender.org> | 2020-04-03 20:15:01 +0300 |
|---|---|---|
| committer | Dalai Felinto <dalai@blender.org> | 2020-04-03 20:27:46 +0300 |
| commit | d138cbfb47e379edc1ee915a8c6ff65b01f000d6 (patch) | |
| tree | f4773ecce897c32eaf75295dd5f6a8e730d471ed /source/blender/editors/interface/interface_region_color_picker.c | |
| parent | b0c1184875d39abac4a65a5d20e263ea6d841009 (diff) | |
Code Quality: Replace for loops with LISTBASE_FOREACH
Note this only changes cases where the variable was declared inside
the for loop. To handle it outside as well is a different challenge.
Differential Revision: https://developer.blender.org/D7320
Diffstat (limited to 'source/blender/editors/interface/interface_region_color_picker.c')
| -rw-r--r-- | source/blender/editors/interface/interface_region_color_picker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/editors/interface/interface_region_color_picker.c b/source/blender/editors/interface/interface_region_color_picker.c index ecdcd575d5c..f9873f8b96f 100644 --- a/source/blender/editors/interface/interface_region_color_picker.c +++ b/source/blender/editors/interface/interface_region_color_picker.c @@ -332,7 +332,7 @@ static void ui_popup_close_cb(bContext *UNUSED(C), void *bt1, void *UNUSED(arg)) static void ui_colorpicker_hide_reveal(uiBlock *block, enum ePickerType colormode) { /* tag buttons */ - for (uiBut *bt = block->buttons.first; bt; bt = bt->next) { + LISTBASE_FOREACH (uiBut *, bt, &block->buttons) { if ((bt->func == ui_colorpicker_rna_cb) && (bt->type == UI_BTYPE_NUM_SLIDER) && (bt->rnaindex != 3)) { /* RGB sliders (color circle and alpha are always shown) */ |