cogl-blend-string: Don't split combined blend statements into two
authorNeil Roberts <neil@linux.intel.com>
Wed, 12 May 2010 16:56:25 +0000 (17:56 +0100)
committerNeil Roberts <neil@linux.intel.com>
Mon, 17 May 2010 15:31:28 +0000 (16:31 +0100)
When a single statement is used to specify the factors for both the
RGB and alpha parts it previously split up the statement into
two. This works but it ends up unnecessarily using glBlendFuncSeparate
when glBlendFunc would suffice.

For example, the blend statement

 RGBA = ADD(SRC_COLOR*(SRC_COLOR), DST_COLOR*(1-SRC_COLOR))

would get split into the two statements

 RGBA = ADD(SRC_COLOR*(SRC_COLOR[RGB]), DST_COLOR*(1-SRC_COLOR[RGB]))
 A    = ADD(SRC_COLOR*(SRC_COLOR[A]), DST_COLOR*(1-SRC_COLOR[A]))

That translates to:

 glBlendFuncSeparate (GL_SRC_COLOR, GL_ONE_MINUS_SRC_COLOR,
                      GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);

This patch makes it so that arg_to_gl_blend_factor can handle the
combined RGBA mask instead. That way the single statement gets
translated to the equivalent call:

 glBlendFunc (GL_SRC_COLOR, GL_ONE_MINUS_SRC_COLOR);

clutter/cogl/cogl/cogl-material.c

index 462c644..d13988c 100644 (file)
@@ -563,7 +563,7 @@ arg_to_gl_blend_factor (CoglBlendStringArgument *arg)
   else if (arg->factor.source.info->type ==
            COGL_BLEND_STRING_COLOR_SOURCE_SRC_COLOR)
     {
-      if (arg->factor.source.mask == COGL_BLEND_STRING_CHANNEL_MASK_RGB)
+      if (arg->factor.source.mask != COGL_BLEND_STRING_CHANNEL_MASK_ALPHA)
         {
           if (arg->factor.source.one_minus)
             return GL_ONE_MINUS_SRC_COLOR;
@@ -581,7 +581,7 @@ arg_to_gl_blend_factor (CoglBlendStringArgument *arg)
   else if (arg->factor.source.info->type ==
            COGL_BLEND_STRING_COLOR_SOURCE_DST_COLOR)
     {
-      if (arg->factor.source.mask == COGL_BLEND_STRING_CHANNEL_MASK_RGB)
+      if (arg->factor.source.mask != COGL_BLEND_STRING_CHANNEL_MASK_ALPHA)
         {
           if (arg->factor.source.one_minus)
             return GL_ONE_MINUS_DST_COLOR;
@@ -600,7 +600,7 @@ arg_to_gl_blend_factor (CoglBlendStringArgument *arg)
   else if (arg->factor.source.info->type ==
            COGL_BLEND_STRING_COLOR_SOURCE_CONSTANT)
     {
-      if (arg->factor.source.mask == COGL_BLEND_STRING_CHANNEL_MASK_RGB)
+      if (arg->factor.source.mask != COGL_BLEND_STRING_CHANNEL_MASK_ALPHA)
         {
           if (arg->factor.source.one_minus)
             return GL_ONE_MINUS_CONSTANT_COLOR;
@@ -651,7 +651,6 @@ cogl_material_set_blend (CoglHandle handle,
 {
   CoglMaterial *material;
   CoglBlendStringStatement statements[2];
-  CoglBlendStringStatement split[2];
   CoglBlendStringStatement *rgb;
   CoglBlendStringStatement *a;
   GError *internal_error = NULL;
@@ -679,13 +678,8 @@ cogl_material_set_blend (CoglHandle handle,
       return FALSE;
     }
 
-  if (statements[0].mask == COGL_BLEND_STRING_CHANNEL_MASK_RGBA)
-    {
-      _cogl_blend_string_split_rgba_statement (statements,
-                                               &split[0], &split[1]);
-      rgb = &split[0];
-      a = &split[1];
-    }
+  if (count == 1)
+    rgb = a = statements;
   else
     {
       rgb = &statements[0];