diff options
author | Clément Foucault <foucault.clem@gmail.com> | 2018-04-06 15:30:20 +0300 |
---|---|---|
committer | Clément Foucault <foucault.clem@gmail.com> | 2018-04-06 15:30:20 +0300 |
commit | 39b654f4ff123e3e9bc345b25d4883df4de4ad5e (patch) | |
tree | 530e6c669f6a74ca1375841a2e05b11175c72d5b /source/blender | |
parent | 21113ad8343c8ce80508aac08c5afbc2aa9dd2e7 (diff) |
UI: Perf: Use widget base batching
Overall 10% more performance on general UI drawing time.
This commit can introduce ordering problem on some elements.
In this case you need to flush the widget cache to ensure the element that
is going to be drawn is drawn on top of any widget base.
To flush the cache use UI_widgetbase_draw_cache_flush.
This is already done for BLF and Icons.
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/blenfont/intern/blf_font.c | 5 | ||||
-rw-r--r-- | source/blender/editors/interface/interface.c | 2 | ||||
-rw-r--r-- | source/blender/editors/interface/interface_icons.c | 14 |
3 files changed, 20 insertions, 1 deletions
diff --git a/source/blender/blenfont/intern/blf_font.c b/source/blender/blenfont/intern/blf_font.c index 07e568dd279..cc7eb336b21 100644 --- a/source/blender/blenfont/intern/blf_font.c +++ b/source/blender/blenfont/intern/blf_font.c @@ -58,6 +58,8 @@ #include "BIF_gl.h" #include "BLF_api.h" +#include "UI_interface.h" + #include "GPU_immediate.h" #include "GPU_matrix.h" #include "GPU_batch.h" @@ -184,6 +186,9 @@ void blf_batch_draw(void) glEnable(GL_BLEND); glBlendFuncSeparate(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA, GL_ONE, GL_ONE_MINUS_SRC_ALPHA); + /* We need to flush widget base first to ensure correct ordering. */ + UI_widgetbase_draw_cache_flush(); + BLI_assert(g_batch.font->tex_bind_state != 0); /* must still be valid */ glActiveTexture(GL_TEXTURE0); glBindTexture(GL_TEXTURE_2D, g_batch.font->tex_bind_state); diff --git a/source/blender/editors/interface/interface.c b/source/blender/editors/interface/interface.c index 0c786874180..12ee1ad35a1 100644 --- a/source/blender/editors/interface/interface.c +++ b/source/blender/editors/interface/interface.c @@ -1429,6 +1429,7 @@ void UI_block_draw(const bContext *C, uiBlock *block) BLF_batch_draw_begin(); UI_icon_draw_cache_begin(); + UI_widgetbase_draw_cache_begin(); /* widgets */ for (but = block->buttons.first; but; but = but->next) { @@ -1442,6 +1443,7 @@ void UI_block_draw(const bContext *C, uiBlock *block) } } + UI_widgetbase_draw_cache_end(); UI_icon_draw_cache_end(); BLF_batch_draw_end(); diff --git a/source/blender/editors/interface/interface_icons.c b/source/blender/editors/interface/interface_icons.c index 9fbb5efdcf3..682eac6a352 100644 --- a/source/blender/editors/interface/interface_icons.c +++ b/source/blender/editors/interface/interface_icons.c @@ -1009,6 +1009,9 @@ static void icon_draw_rect(float x, float y, int w, int h, float UNUSED(aspect), rect = ima->rect; } + /* We need to flush widget base first to ensure correct ordering. */ + UI_widgetbase_draw_cache_flush(); + /* draw */ IMMDrawPixelsTexState state = immDrawPixelsTexSetup(GPU_SHADER_2D_IMAGE_COLOR); immDrawPixelsTex(&state, draw_x, draw_y, draw_w, draw_h, GL_RGBA, GL_UNSIGNED_BYTE, GL_NEAREST, rect, @@ -1049,6 +1052,12 @@ static void icon_draw_cache_flush_ex(void) if (g_icon_draw_cache.calls == 0) return; + /* We need to flush widget base first to ensure correct ordering. */ + glBlendFuncSeparate(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA, GL_ONE, GL_ONE_MINUS_SRC_ALPHA); + UI_widgetbase_draw_cache_flush(); + + glBlendFunc(GL_ONE, GL_ONE_MINUS_SRC_ALPHA); + glActiveTexture(GL_TEXTURE0); glBindTexture(GL_TEXTURE_2D, icongltex.id); @@ -1078,7 +1087,6 @@ void UI_icon_draw_cache_end(void) return; glEnable(GL_BLEND); - glBlendFunc(GL_ONE, GL_ONE_MINUS_SRC_ALPHA); icon_draw_cache_flush_ex(); @@ -1125,6 +1133,10 @@ static void icon_draw_texture( return; } + /* We need to flush widget base first to ensure correct ordering. */ + glBlendFuncSeparate(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA, GL_ONE, GL_ONE_MINUS_SRC_ALPHA); + UI_widgetbase_draw_cache_flush(); + float x1, x2, y1, y2; x1 = ix * icongltex.invw; |