On Mon, May 8, 2017 at 5:29 PM, Ilia Mirkin <[email protected]> wrote: > On Mon, May 8, 2017 at 5:27 PM, Rob Clark <[email protected]> wrote: >> On Mon, May 8, 2017 at 4:57 PM, Marek Olšák <[email protected]> wrote: >>> I personally find the assertions unnecessary because the bug has been >>> discovered and fixed by this patch, and this code is unlikely to >>> change. Anyway: >>> >>> Reviewed-by: Marek Olšák <[email protected]> >> >> I don't suppose there is something like debug_assert() (ie. only >> defined for --enable-debug builds) for mesa code? That was really >> what I wanted there. Maybe I'll just put #ifdef DEBUG / #endif around >> it.. > > Yeah. It's called "assert" :)
oh, heh, yeah we are injecting -DNDEBUG=1 for non-debug builds.. so I'll just leave it as-is BR, -R _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
