Remove deleted list for palette colors
authorCampbell Barton <ideasman42@gmail.com>
Thu, 19 Mar 2015 19:07:22 +0000 (06:07 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 19 Mar 2015 19:08:38 +0000 (06:08 +1100)
was used because of UI memory access only.

source/blender/blenkernel/BKE_paint.h
source/blender/blenkernel/intern/paint.c
source/blender/blenloader/intern/readfile.c
source/blender/editors/interface/interface_handlers.c
source/blender/editors/interface/interface_templates.c
source/blender/makesdna/DNA_brush_types.h
source/blender/makesrna/intern/rna_palette.c

index 58c0b49506285ac84d46c831ffddbb94ce83c2ec..9076eada3120111b8fa01ad64190857ff326dc09 100644 (file)
@@ -102,9 +102,7 @@ void                 BKE_palette_free(struct Palette *palette);
 struct Palette      *BKE_palette_add(struct Main *bmain, const char *name);
 struct PaletteColor *BKE_palette_color_add(struct Palette *palette);
 bool                 BKE_palette_is_empty(const struct Palette *palette);
-void                 BKE_palette_color_remove_ex(struct Palette *palette, struct PaletteColor *color, bool use_free);
 void                 BKE_palette_color_remove(struct Palette *palette, struct PaletteColor *color);
-void                 BKE_palette_cleanup(struct Palette *palette);
 void                 BKE_palette_clear(struct Palette *palette);
 
 /* paint curves */
index 2082066d395cfc5fa970f6a4b44945c044b7b8ec..c3c88389e11ea4796f567fdb5145f6b2d2d8e82c 100644 (file)
@@ -314,7 +314,7 @@ void BKE_paint_curve_set(Brush *br, PaintCurve *pc)
 }
 
 /* remove colour from palette. Must be certain color is inside the palette! */
-void BKE_palette_color_remove_ex(Palette *palette, PaletteColor *color, bool use_free)
+void BKE_palette_color_remove(Palette *palette, PaletteColor *color)
 {
        if (BLI_listbase_count_ex(&palette->colors, palette->active_color) == palette->active_color) {
                palette->active_color--;
@@ -326,32 +326,15 @@ void BKE_palette_color_remove_ex(Palette *palette, PaletteColor *color, bool use
                palette->active_color = 0;
        }
 
-       if (use_free) {
-               MEM_freeN(color);
-       }
-       else {
-               BLI_addhead(&palette->deleted, color);
-       }
-}
-
-void BKE_palette_color_remove(Palette *palette, PaletteColor *color)
-{
-       BKE_palette_color_remove_ex(palette, color, false);
+       MEM_freeN(color);
 }
 
 void BKE_palette_clear(Palette *palette)
 {
        BLI_freelistN(&palette->colors);
-       BLI_freelistN(&palette->deleted);
        palette->active_color = 0;
 }
 
-void BKE_palette_cleanup(Palette *palette)
-{
-       BLI_freelistN(&palette->deleted);
-}
-
-
 Palette *BKE_palette_add(Main *bmain, const char *name)
 {
        Palette *palette;
index 6b4d45a85dcdec3a996d5199166c763eb7bb73a2..639ee646a67250acb0bdf17e7be25659a27d138c 100644 (file)
@@ -1950,7 +1950,6 @@ static void direct_link_palette(FileData *fd, Palette *palette)
 {
        /* palette itself has been read */
        link_list(fd, &palette->colors);
-       BLI_listbase_clear(&palette->deleted);
 }
 
 static void lib_link_paint_curve(FileData *UNUSED(fd), Main *main)
index a6ff5bb3ad34f16256c50f5369c55931be9790cb..71b7b95208bf63e74d232c8cc2c7acc5cd96c300 100644 (file)
@@ -4494,6 +4494,12 @@ static int ui_do_but_COLOR(bContext *C, uiBut *but, uiHandleButtonData *data, co
                        BKE_palette_color_remove(palette, color);
 
                        button_activate_state(C, but, BUTTON_STATE_EXIT);
+
+                       /* this is risky. it works OK for now,
+                        * but if it gives trouble we should delay execution */
+                       but->rnapoin = PointerRNA_NULL;
+                       but->rnaprop = NULL;
+
                        return WM_UI_HANDLER_BREAK;
                }
        }
index b3c31a1a644ad641caf15991ef3d641d565b748d..7db734ee9197565fc6eecedb4c00e3f7ab600b68 100644 (file)
@@ -2398,9 +2398,6 @@ void uiTemplatePalette(uiLayout *layout, PointerRNA *ptr, const char *propname,
 
        palette = cptr.data;
 
-       /* first delete any pending colors */
-       BKE_palette_cleanup(palette);
-
        color = palette->colors.first;
 
        col = uiLayoutColumn(layout, true);
index 89c8316002b5d3b09276fb0e633d5e5a1fef9c1b..dd66acf27815f88577fd723b2302d3c69ae32020 100644 (file)
@@ -151,7 +151,6 @@ typedef struct Palette
 
        /* pointer to individual colours */
        ListBase colors;
-       ListBase deleted;
 
        int active_color;
        int pad;
index 7405df967a0797ccea320f7105b0d5bf77538c36..8cbb57fde2c960049de6aaaaedb5359bda456c9d 100644 (file)
@@ -55,7 +55,7 @@ static void rna_Palette_color_remove(Palette *palette, ReportList *reports, Poin
                return;
        }
 
-       BKE_palette_color_remove_ex(palette, color, true);
+       BKE_palette_color_remove(palette, color);
 
        RNA_POINTER_INVALIDATE(color_ptr);
 }