[Mesa-dev] [PATCH 0/9] i965: A few tiny trivial cleanups

Tapani Pälli tapani.palli at intel.com
Mon May 29 06:46:54 UTC 2017


LGTM
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

On 05/27/2017 01:15 AM, Chad Versace wrote:
> I was hacking on the i965/DRI glue, and on brw_surface_formats.c, and
> I found several little cleanups to make the code nicer. All these
> patches are super small.
> 
> Chad Versace (9):
>    i965/dri: Rewrite comment for intelCreateBuffer
>    i965/dri: Combine declaration and assignment in intelCreateBuffer
>    i965: Rename intel_create_renderbuffer
>    i965: Fix type of intel_update_image_buffers::format
>    i965: Move an 'i' declaration into its 'for' loop
>    i965: Add whitespace in intel_update_image_buffers()
>    i965: Document type of GLuint __DRIimage::format
>    i965: Move func to right comment block in brw_context.h
>    i965: Fix type of brw_context::render_target_format[]
> 
>   src/mesa/drivers/dri/i965/brw_blorp.c            |  4 ++--
>   src/mesa/drivers/dri/i965/brw_context.c          |  7 ++++---
>   src/mesa/drivers/dri/i965/brw_context.h          |  4 ++--
>   src/mesa/drivers/dri/i965/brw_wm_surface_state.c |  4 ++--
>   src/mesa/drivers/dri/i965/intel_fbo.c            | 11 ++++++-----
>   src/mesa/drivers/dri/i965/intel_fbo.h            |  2 +-
>   src/mesa/drivers/dri/i965/intel_image.h          |  2 +-
>   src/mesa/drivers/dri/i965/intel_screen.c         | 13 ++++++++-----
>   8 files changed, 26 insertions(+), 21 deletions(-)
> 


More information about the mesa-dev mailing list