[Cogl] [PATCH] Clearly define 3 progends that own the frag+vertends

Neil Roberts neil at linux.intel.com
Wed Sep 26 06:40:46 PDT 2012


Looks good to me, apart from one minor nitpick:

Robert Bragg writes:

>    _cogl_use_fragment_program (0, COGL_PIPELINE_PROGRAM_TYPE_FIXED);
> -  return TRUE;
> +  return;
>  }

I think it's pretty unlikely the program will forget to return out of
the end this function so we can probably remove the redundant return
statement altogether.. :p There are a few places that are doing this.

Reviewed-by: Neil Roberts <neil at linux.intel.com>

- Neil


More information about the Cogl mailing list