commit 0b01e3cf35f8429247e3a72c77bdcdc678a40f0c
author Alex Szpakowski <slime73@gmail.com> Sat Aug 17 22:26:33 2019 -0300
committer Alex Szpakowski <slime73@gmail.com> Sat Aug 17 22:26:33 2019 -0300
tree 5d29c48bd4c628b8c97aac71500f80f65a84cfa3
parent fd9b2e72575c4abac112e38d149d6a3399802569

render: avoid a couple redundant memcmp calls in all drawing functions. Improves performance slightly.

src/render/SDL_render.c

diff --git a/src/render/SDL_render.c b/src/render/SDL_render.c
index 6e7ec61..760adb0 100644
--- a/src/render/SDL_render.c
+++ b/src/render/SDL_render.c
@@ -410,14 +410,17 @@
 static int
 PrepQueueCmdDraw(SDL_Renderer *renderer, const Uint8 r, const Uint8 g, const Uint8 b, const Uint8 a)
 {
+    /* Guarantee that the draw color, viewport, and clip rect are set before
+     * draws, so the backends don't have to worry about that state not being
+     * valid at draw time. */
     int retval = 0;
-    if (retval == 0) {
+    if (retval == 0 && !renderer->color_queued) {
         retval = QueueCmdSetDrawColor(renderer, r, g, b, a);
     }
-    if (retval == 0) {
+    if (retval == 0 && !renderer->viewport_queued) {
         retval = QueueCmdSetViewport(renderer);
     }
-    if (retval == 0) {
+    if (retval == 0 && !renderer->cliprect_queued) {
         retval = QueueCmdSetClipRect(renderer);
     }
     return retval;