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