diff options
author | Clément Foucault <foucault.clem@gmail.com> | 2017-05-30 23:28:47 +0300 |
---|---|---|
committer | Clément Foucault <foucault.clem@gmail.com> | 2017-05-30 23:30:16 +0300 |
commit | e7fb013a60dd91caef1b1dd80f52b3996f06a21c (patch) | |
tree | ca9ec934203b7cc5e48f2fbc65b982487be91323 /source/blender/draw | |
parent | ff0210319480ac9a77897bed716088a0bc18cd35 (diff) |
Draw Manager: Add SceneLayerEngineData and use callbacks to clear runtime engine data
Diffstat (limited to 'source/blender/draw')
-rw-r--r-- | source/blender/draw/DRW_engine.h | 2 | ||||
-rw-r--r-- | source/blender/draw/intern/DRW_render.h | 6 | ||||
-rw-r--r-- | source/blender/draw/intern/draw_manager.c | 49 | ||||
-rw-r--r-- | source/blender/draw/modes/object_mode.c | 2 |
4 files changed, 37 insertions, 22 deletions
diff --git a/source/blender/draw/DRW_engine.h b/source/blender/draw/DRW_engine.h index c5cf029febb..0aca41a970d 100644 --- a/source/blender/draw/DRW_engine.h +++ b/source/blender/draw/DRW_engine.h @@ -80,8 +80,6 @@ void DRW_draw_depth_loop( struct Depsgraph *graph, struct ARegion *ar, struct View3D *v3d); -void DRW_object_engine_data_free(struct Object *ob); - /* This is here because GPUViewport needs it */ void DRW_pass_free(struct DRWPass *pass); diff --git a/source/blender/draw/intern/DRW_render.h b/source/blender/draw/intern/DRW_render.h index 695027c5de6..7285b3c3311 100644 --- a/source/blender/draw/intern/DRW_render.h +++ b/source/blender/draw/intern/DRW_render.h @@ -320,8 +320,12 @@ bool DRW_viewport_is_persp_get(void); struct DefaultFramebufferList *DRW_viewport_framebuffer_list_get(void); struct DefaultTextureList *DRW_viewport_texture_list_get(void); +/* SceneLayers */ +void **DRW_scene_layer_engine_data_get(DrawEngineType *engine_type, void (*callback)(void *storage)); + /* Objects */ -void **DRW_object_engine_data_get(Object *ob, DrawEngineType *det); +void **DRW_object_engine_data_get( + Object *ob, DrawEngineType *engine_type, void (*callback)(void *storage)); struct LampEngineData *DRW_lamp_engine_data_get(Object *ob, struct RenderEngineType *engine_type); void DRW_lamp_engine_data_free(struct LampEngineData *led); diff --git a/source/blender/draw/intern/draw_manager.c b/source/blender/draw/intern/draw_manager.c index cc2ad3e3017..cf882717f8a 100644 --- a/source/blender/draw/intern/draw_manager.c +++ b/source/blender/draw/intern/draw_manager.c @@ -2246,17 +2246,37 @@ DefaultTextureList *DRW_viewport_texture_list_get(void) /* -------------------------------------------------------------------- */ +/** \name SceneLayers (DRW_scenelayer) + * \{ */ + +void **DRW_scene_layer_engine_data_get(DrawEngineType *engine_type, void (*callback)(void *storage)) +{ + SceneLayerEngineData *sled; + + for (sled = DST.draw_ctx.sl->drawdata.first; sled; sled = sled->next) { + if (sled->engine_type == engine_type) { + return &sled->storage; + } + } + + sled = MEM_callocN(sizeof(SceneLayerEngineData), "SceneLayerEngineData"); + sled->engine_type = engine_type; + sled->free = callback; + BLI_addtail(&DST.draw_ctx.sl->drawdata, sled); + + return &sled->storage; +} + +/** \} */ + + +/* -------------------------------------------------------------------- */ /** \name Objects (DRW_object) * \{ */ -typedef struct ObjectEngineData { - struct ObjectEngineData *next, *prev; - DrawEngineType *engine_type; - void *storage; -} ObjectEngineData; - -void **DRW_object_engine_data_get(Object *ob, DrawEngineType *engine_type) +void **DRW_object_engine_data_get( + Object *ob, DrawEngineType *engine_type, void (*callback)(void *storage)) { ObjectEngineData *oed; @@ -2268,22 +2288,14 @@ void **DRW_object_engine_data_get(Object *ob, DrawEngineType *engine_type) oed = MEM_callocN(sizeof(ObjectEngineData), "ObjectEngineData"); oed->engine_type = engine_type; + oed->free = callback; BLI_addtail(&ob->drawdata, oed); return &oed->storage; } -void DRW_object_engine_data_free(Object *ob) -{ - for (ObjectEngineData *oed = ob->drawdata.first; oed; oed = oed->next) { - if (oed->storage) { - MEM_freeN(oed->storage); - } - } - - BLI_freelistN(&ob->drawdata); -} - +/* XXX There is definitly some overlap between this and DRW_object_engine_data_get. + * We should get rid of one of the two. */ LampEngineData *DRW_lamp_engine_data_get(Object *ob, RenderEngineType *engine_type) { BLI_assert(ob->type == OB_LAMP); @@ -2291,6 +2303,7 @@ LampEngineData *DRW_lamp_engine_data_get(Object *ob, RenderEngineType *engine_ty Scene *scene = DST.draw_ctx.scene; /* TODO Dupliobjects */ + /* TODO Should be per scenelayer */ return GPU_lamp_engine_data_get(scene, ob, NULL, engine_type); } diff --git a/source/blender/draw/modes/object_mode.c b/source/blender/draw/modes/object_mode.c index 59debb09298..205a0c16613 100644 --- a/source/blender/draw/modes/object_mode.c +++ b/source/blender/draw/modes/object_mode.c @@ -1071,7 +1071,7 @@ static void DRW_shgroup_lamp(OBJECT_StorageList *stl, Object *ob, SceneLayer *sl int theme_id = DRW_object_wire_theme_get(ob, sl, &color); static float zero = 0.0f; - float **la_mats = (float **)DRW_object_engine_data_get(ob, &draw_engine_object_type); + float **la_mats = (float **)DRW_object_engine_data_get(ob, &draw_engine_object_type, NULL); if (*la_mats == NULL) { /* we need 2 matrices */ *la_mats = MEM_mallocN(sizeof(float) * 16 * 2, "Lamp Object Mode Matrices"); |