[Mesa-dev] [PATCH 1/4] mesa: remove spurious flush in _mesa_DepthRange()

[Brian Paul]

On 06/22/2017 08:35 AM, Samuel Pitoiset wrote:
> I don't think this is actually required, if the depth range
> values are different from the ones stored in the context, we
> already flush and trigger _NEW_VIEWPORT in
> set_depth_range_no_notify().
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>   src/mesa/main/viewport.c | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/src/mesa/main/viewport.c b/src/mesa/main/viewport.c
> index 5529f55ee71..a4097efd6f0 100644
> --- a/src/mesa/main/viewport.c
> +++ b/src/mesa/main/viewport.c
> @@ -274,8 +274,6 @@ _mesa_DepthRange(GLclampd nearval, GLclampd farval)
>      unsigned i;
>      GET_CURRENT_CONTEXT(ctx);
>
> -   FLUSH_VERTICES(ctx, 0);
> -
>      if (MESA_VERBOSE&VERBOSE_API)
>         _mesa_debug(ctx, "glDepthRange %f %f\n", nearval, farval);
>
>

Reviewed-by: Brian Paul <brianp at vmware.com>