On Mon, Mar 4, 2019 at 1:38 PM Alyssa Rosenzweig <aly...@rosenzweig.io> wrote:
>
> >          unsigned transient_count = 
> > ctx->transient_pools[ctx->cmdstream_i].entry_index*ctx->transient_pools[0].entry_size
> >  + ctx->transient_pools[ctx->cmdstream_i].entry_offset;
> > -     printf("Uploaded transient %d bytes\n", transient_count);
> > +     //printf("Uploaded transient %d bytes\n", transient_count);
>
> This will raised an unused variable warning for transient_count, better
> comment both or neither.

Better yet, use a debug print so it can be enabled via an env var. Or
if it's not that useful, just remove it.

>
> Other than that, Reviewed-by: Alyssa Rosenzweig <aly...@rosenzweig.io>
>
> I wonder if this will fix some of the memory leaks on non-DRM as well.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to