Re: [Mesa-dev] [PATCH 8/8] i965: Drop mt->levels[].width/height.

2014-02-14 Thread Kenneth Graunke
On 02/14/2014 03:00 PM, Eric Anholt wrote: > It often confused people because it was unclear on whether it was the > physical or logical, and people needed the other one as well. We can > recompute it trivially using the minify() macro, clarifying which value is > being used and making getting the

Re: [Mesa-dev] [PATCH 8/8] i965: Drop mt->levels[].width/height.

2014-02-14 Thread Chris Forbes
This is a nice improvement. Reviewed-by: Chris Forbes On Sat, Feb 15, 2014 at 12:00 PM, Eric Anholt wrote: > It often confused people because it was unclear on whether it was the > physical or logical, and people needed the other one as well. We can > recompute it trivially using the minify()

[Mesa-dev] [PATCH 8/8] i965: Drop mt->levels[].width/height.

2014-02-14 Thread Eric Anholt
It often confused people because it was unclear on whether it was the physical or logical, and people needed the other one as well. We can recompute it trivially using the minify() macro, clarifying which value is being used and making getting the other value obvious. --- src/mesa/drivers/dri/i96