[gegl/gsoc2011-opencl-2: 13/19] Better error management in Brightness-Contrast OpenCL version

[Victor Matheus de Araujo Oliveira <vmaolive src gnome org>]

commit 0ec8c85ab1efa21467b2897cb37da53aab23583e
Author: Victor Oliveira <victormatheus gmail com>
Date:   Fri Nov 18 17:02:23 2011 -0200

    Better error management in Brightness-Contrast OpenCL version

 operations/common/brightness-contrast.c |   24 +++++++++++++++++-------
 1 files changed, 17 insertions(+), 7 deletions(-)
---
diff --git a/operations/common/brightness-contrast.c b/operations/common/brightness-contrast.c
index a11ec42..e32ba54 100644
--- a/operations/common/brightness-contrast.c
+++ b/operations/common/brightness-contrast.c
@@ -136,12 +136,13 @@ cl_process (GeglOperation       *op,
   /* Retrieve a pointer to GeglChantO structure which contains all the
    * chanted properties
    */
+
   GeglChantO *o = GEGL_CHANT_PROPERTIES (op);
 
   gfloat brightness = o->brightness;
   gfloat contrast   = o->contrast;
 
-  cl_int errcode;
+  cl_int errcode = 0;
 
   if (!cl_data)
     {
@@ -149,17 +150,26 @@ cl_process (GeglOperation       *op,
       cl_data = gegl_cl_compile_and_build (kernel_source, kernel_name);
     }
 
-  CL_SAFE_CALL(errcode = gegl_clSetKernelArg(cl_data->kernel[0], 0, sizeof(cl_mem),   (void*)&in_tex));
-  CL_SAFE_CALL(errcode = gegl_clSetKernelArg(cl_data->kernel[0], 1, sizeof(cl_mem),   (void*)&out_tex));
-  CL_SAFE_CALL(errcode = gegl_clSetKernelArg(cl_data->kernel[0], 2, sizeof(cl_float), (void*)&brightness));
-  CL_SAFE_CALL(errcode = gegl_clSetKernelArg(cl_data->kernel[0], 3, sizeof(cl_float), (void*)&contrast));
+  if (!cl_data) return 1;
+
+  CL_SAFE_CALL(errcode |= gegl_clSetKernelArg(cl_data->kernel[0], 0, sizeof(cl_mem),   (void*)&in_tex));
+  CL_SAFE_CALL(errcode |= gegl_clSetKernelArg(cl_data->kernel[0], 1, sizeof(cl_mem),   (void*)&out_tex));
+  CL_SAFE_CALL(errcode |= gegl_clSetKernelArg(cl_data->kernel[0], 2, sizeof(cl_float), (void*)&brightness));
+  CL_SAFE_CALL(errcode |= gegl_clSetKernelArg(cl_data->kernel[0], 3, sizeof(cl_float), (void*)&contrast));
 
-  CL_SAFE_CALL(errcode = gegl_clEnqueueNDRangeKernel(gegl_cl_get_command_queue (),
+  CL_SAFE_CALL(errcode |= gegl_clEnqueueNDRangeKernel(gegl_cl_get_command_queue (),
                                                      cl_data->kernel[0], 2,
                                                      NULL, global_worksize, NULL,
                                                      0, NULL, NULL) );
 
-  return 0;
+  if (errcode != CL_SUCCESS)
+    {
+      g_log(G_LOG_DOMAIN, G_LOG_LEVEL_INFO, "[OpenCL] Error in Brightness-Constrast Kernel\n");
+      return errcode;
+    }
+
+  g_log(G_LOG_DOMAIN, G_LOG_LEVEL_INFO, "[OpenCL] Running Brightness-Constrast Kernel in region (%d %d %d %d)", roi->x, roi->y, roi->width, roi->height);
+  return errcode;
 }
 
 /*