On Thu, Feb 23, 2012 at 3:12 PM, Vincent Lejeune <v...@ovi.com> wrote:
> ---
>  src/mesa/main/mtypes.h    |   10 ++++++++++
>  src/mesa/main/shaderapi.c |    1 +
>  2 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
> index 9200f3f..447d432 100644
> --- a/src/mesa/main/mtypes.h
> +++ b/src/mesa/main/mtypes.h
> @@ -2392,6 +2392,14 @@ struct gl_shader_state
>    GLbitfield Flags;                    /**< Mask of GLSL_x flags */
>  };
>
> +enum packing_constraint
> +{
> +   NONE, /** no limitation (scalar architecture : swrast, nouveau) */
> +   SMOOTH_NOPERSPECTIVE_MIXED, /** Smooth and noperspective varying can be 
> mixed but not with flat (i965) */
> +   NO_MIXED_INTERPOLATION, /** Dont mix different interpolation (r300 and 
> r600) */
> +   AVOID_PACKING, /** Disable packing */
> +};

These enum values should have a common prefix like
"PACKING_CONSTRAINT_" like we do for other enums.  See
FRAG_DEPTH_LAYOUT, for example.

And Doxygen comments after an item should start with /**<

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to