[gtk+/wip/baedert/gl: 82/94] gl renderer: Default to 0px clip and border radius



commit 5a2c6cd92b9034e55cc13a92fc13856f1e3f46a3
Author: Timm Bäder <mail baedert org>
Date:   Fri Dec 8 22:19:52 2017 +0100

    gl renderer: Default to 0px clip and border radius
    
    Finally fix the division by 0 in the clip code.

 gsk/gl/gskglrenderer.c                |   62 ++++++++++++++++----------------
 gsk/resources/glsl/gl3_common.fs.glsl |    7 +++-
 2 files changed, 36 insertions(+), 33 deletions(-)
---
diff --git a/gsk/gl/gskglrenderer.c b/gsk/gl/gskglrenderer.c
index 40020de..e572420 100644
--- a/gsk/gl/gskglrenderer.c
+++ b/gsk/gl/gskglrenderer.c
@@ -106,8 +106,8 @@ rounded_rect_to_floats (const GskRoundedRect *rect,
 
   for (i = 0; i < 4; i ++)
     {
-      corner_widths[i] = MAX (rect->corner[i].width, 1);
-      corner_heights[i] = MAX (rect->corner[i].height, 1);
+      corner_widths[i] = rect->corner[i].width;
+      corner_heights[i] = rect->corner[i].height;
     }
 }
 
@@ -409,48 +409,48 @@ render_border_node (GskGLRenderer   *self,
 
   /* Top left */
   if (widths[3] > 0)
-    sizes[0].w = MAX (MAX (widths[3], rounded_outline->corner[0].width), 1);
+    sizes[0].w = MAX (widths[3], rounded_outline->corner[0].width);
   else
-    sizes[0].w = 1;
+    sizes[0].w = 0;
 
   if (widths[0] > 0)
-    sizes[0].h = MAX (MAX (widths[0], rounded_outline->corner[0].height), 1);
+    sizes[0].h = MAX (widths[0], rounded_outline->corner[0].height);
   else
-    sizes[0].h = 1;
+    sizes[0].h = 0;
 
   /* Top right */
   if (widths[1] > 0)
-    sizes[1].w = MAX (MAX (widths[1], rounded_outline->corner[1].width), 1);
+    sizes[1].w = MAX (widths[1], rounded_outline->corner[1].width);
   else
-    sizes[1].w = 1;
+    sizes[1].w = 0;
 
   if (widths[0] > 0)
-    sizes[1].h = MAX (MAX (widths[0], rounded_outline->corner[1].height), 1);
+    sizes[1].h = MAX (widths[0], rounded_outline->corner[1].height);
   else
-    sizes[1].h = 1;
+    sizes[1].h = 0;
 
   /* Bottom right */
   if (widths[1] > 0)
     sizes[2].w = MAX (widths[1], rounded_outline->corner[2].width);
   else
-    sizes[2].w = 1;
+    sizes[2].w = 0;
 
   if (widths[2] > 0)
     sizes[2].h = MAX (widths[2], rounded_outline->corner[2].height);
   else
-    sizes[2].h = 1;
+    sizes[2].h = 0;
 
 
   /* Bottom left */
   if (widths[3] > 0)
     sizes[3].w = MAX (widths[3], rounded_outline->corner[3].width);
   else
-    sizes[3].w = 1;
+    sizes[3].w = 0;
 
   if (widths[2] > 0)
     sizes[3].h = MAX (widths[2], rounded_outline->corner[3].height);
   else
-    sizes[3].h = 1;
+    sizes[3].h = 0;
 
   if (needs_clip)
     {
@@ -514,7 +514,7 @@ render_border_node (GskGLRenderer   *self,
         { { min_x,              max_y              }, { 0, 0 }, }, /* Lower left */
         { { min_x + sizes[0].w, min_y + sizes[0].h }, { 1, 1 }, }, /* Upper right */
 
-        { { min_x + sizes[3].w, max_y - sizes[2].h }, { 1, 0 }, }, /* Lower right */
+        { { min_x + sizes[3].w, max_y - sizes[3].h }, { 1, 0 }, }, /* Lower right */
         { { min_x,              max_y              }, { 0, 0 }, }, /* Lower left */
         { { min_x + sizes[0].w, min_y + sizes[0].h }, { 1, 1 }, }, /* Upper right */
       }
@@ -1718,28 +1718,28 @@ gsk_gl_renderer_render_ops (GskGLRenderer *self,
           OP_PRINT (" -> Clip (%f, %f, %f, %f) (%f, %f, %f, %f), (%f, %f, %f, %f)",
                     op->clip.bounds.origin.x, op->clip.bounds.origin.y,
                     op->clip.bounds.size.width, op->clip.bounds.size.height,
-                    MAX (op->clip.corner[0].width, 1),
-                    MAX (op->clip.corner[1].width, 1),
-                    MAX (op->clip.corner[2].width, 1),
-                    MAX (op->clip.corner[3].width, 1),
-                    MAX (op->clip.corner[0].height, 1),
-                    MAX (op->clip.corner[1].height, 1),
-                    MAX (op->clip.corner[2].height, 1),
-                    MAX (op->clip.corner[3].height, 1));
+                    op->clip.corner[0].width,
+                    op->clip.corner[1].width,
+                    op->clip.corner[2].width,
+                    op->clip.corner[3].width,
+                    op->clip.corner[0].height,
+                    op->clip.corner[1].height,
+                    op->clip.corner[2].height,
+                    op->clip.corner[3].height);
           glUniform4f (program->clip_location,
                        op->clip.bounds.origin.x, op->clip.bounds.origin.y,
                        op->clip.bounds.size.width, op->clip.bounds.size.height);
 
           glUniform4f (program->clip_corner_widths_location,
-                       MAX (op->clip.corner[0].width, 1),
-                       MAX (op->clip.corner[1].width, 1),
-                       MAX (op->clip.corner[2].width, 1),
-                       MAX (op->clip.corner[3].width, 1));
+                       op->clip.corner[0].width,
+                       op->clip.corner[1].width,
+                       op->clip.corner[2].width,
+                       op->clip.corner[3].width);
           glUniform4f (program->clip_corner_heights_location,
-                       MAX (op->clip.corner[0].height, 1),
-                       MAX (op->clip.corner[1].height, 1),
-                       MAX (op->clip.corner[2].height, 1),
-                       MAX (op->clip.corner[3].height, 1));
+                       op->clip.corner[0].height,
+                       op->clip.corner[1].height,
+                       op->clip.corner[2].height,
+                       op->clip.corner[3].height);
           break;
 
         case OP_CHANGE_SOURCE_TEXTURE:
diff --git a/gsk/resources/glsl/gl3_common.fs.glsl b/gsk/resources/glsl/gl3_common.fs.glsl
index 6bf5f4d..fe360c4 100644
--- a/gsk/resources/glsl/gl3_common.fs.glsl
+++ b/gsk/resources/glsl/gl3_common.fs.glsl
@@ -10,8 +10,8 @@ uniform vec4 u_viewport;
 
 // In GtkSnapshot coordinates
 uniform vec4 u_clip;
-uniform vec4 u_clip_corner_widths  = vec4(1, 1, 1, 1);
-uniform vec4 u_clip_corner_heights = vec4(1, 1, 1, 1);
+uniform vec4 u_clip_corner_widths  = vec4(0, 0, 0, 0);
+uniform vec4 u_clip_corner_heights = vec4(0, 0, 0, 0);
 
 in vec2 vUv;
 
@@ -28,6 +28,9 @@ struct RoundedRect
 float
 ellipsis_dist (vec2 p, vec2 radius)
 {
+  if (radius == vec2(0, 0))
+    return 0.0;
+
   vec2 p0 = p / radius;
   vec2 p1 = 2.0 * p0 / radius;
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]