On Sat, Mar 19, 2016 at 6:07 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote:
> Normally samples == 0 is perfecly valid and means "not multisampling".
> Why is it invalid here?

Because this function never returns 0 and drivers depend on that
behavior. The patch looks good:

Reviewed-by: Marek Olšák <marek.ol...@amd.com>

I would just prefer it if the patch used "NOTE:" instead of "N.B.".

Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to