[Mesa-dev] [PATCH 4/7] mesa: add SEVERITY_NOTIFICATION to default state

Timothy Arceri t_arceri at yahoo.com.au
Sun Nov 22 16:10:51 PST 2015


On Sun, 2015-11-22 at 23:21 +0000, Emil Velikov wrote:
> As per the spec quote:
> 
>     "All messages are initially enabled unless their assigned severity
>     is DEBUG_SEVERITY_LOW"
> 
> We already had MEDIUM and HIGH set, let's toggle NOTIFICATION as well.

Looks like there was a spec bug and one of the items was to reorder table
Table 5.5 but this was missed when the bug was closed.

> 
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  src/mesa/main/errors.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
> index 52dbd50..b9e5f4f 100644
> --- a/src/mesa/main/errors.c
> +++ b/src/mesa/main/errors.c
> @@ -243,8 +243,9 @@ debug_namespace_init(struct gl_debug_namespace *ns)
>     make_empty_list(&ns->Elements);
>  
>     /* Enable all the messages with severity HIGH or MEDIUM by default */
> -   ns->DefaultState = (1 << MESA_DEBUG_SEVERITY_HIGH) |
> -                      (1 << MESA_DEBUG_SEVERITY_MEDIUM);
> +   ns->DefaultState = (1 << MESA_DEBUG_SEVERITY_MEDIUM ) |
> +                      (1 << MESA_DEBUG_SEVERITY_HIGH) |
> +                      (1 << MESA_DEBUG_SEVERITY_NOTIFICATION);
>  }
>  
>  static void


More information about the mesa-dev mailing list