[Mesa-dev] [PATCH 00/15] Clang compiler warning fixes

[Brian Paul]

On 04/11/2017 11:58 AM, Samuel Pitoiset wrote:
> Noticed while building my whole ARB_bindless_texture work to avoid
> missing switch cases, etc. This series doesn't fix all warnings, just
> the easy ones.
>
> Please, review!
> Thanks.

For 1-14, Reviewed-by: Brian Paul <brianp at vmware.com>

-Brian


>
> Samuel Pitoiset (15):
>    virgl: add missing PIPE_CAP_DOUBLES
>    mesa: remove unused _mesa_unmarshal_BindBufferBase()
>    mesa: remove unused clamp_float_to_uint() and clamp_half_to_uint()
>    mesa: remove some unused functions in the perf monitor area
>    draw: remove unused overflow()
>    draw: remove unused wideline_stage()
>    trace: remove some unused trace_dump_tag*() functions
>    softpipe: remove unused get_texel_quad_2d()
>    softpipe: remove unused quad_shade_stage()
>    softpipe: remove unused sp_exec_fragment_shader()
>    llvmpipe: remove unused subpixel_snap() and fixed_to_float()
>    svga: remove unused vmw_dri1_intersect_src_bbox()
>    radeon: remove duplicate 'const' specifier
>    st/mesa: fix wrong comparison in update_framebuffer_state()
>    i965: add missing ir_unop_*/ir_binop_* in visit_leave()
>
>   src/gallium/auxiliary/draw/draw_pipe_vbuf.c        |  8 ----
>   src/gallium/auxiliary/draw/draw_pipe_wide_line.c   |  8 ----
>   src/gallium/drivers/llvmpipe/lp_setup_tri.c        | 12 -----
>   src/gallium/drivers/softpipe/sp_fs_exec.c          |  8 ----
>   src/gallium/drivers/softpipe/sp_quad_fs.c          |  8 ----
>   src/gallium/drivers/softpipe/sp_tex_sample.c       | 17 -------
>   src/gallium/drivers/trace/tr_dump.c                | 52 ----------------------
>   src/gallium/drivers/virgl/virgl_screen.c           |  1 +
>   src/gallium/winsys/svga/drm/vmw_screen_dri.c       | 32 -------------
>   .../dri/i965/brw_fs_channel_expressions.cpp        |  3 ++
>   .../drivers/dri/radeon/radeon_common_context.c     |  2 +-
>   .../drivers/dri/radeon/radeon_common_context.h     |  2 +-
>   src/mesa/main/marshal.c                            |  8 ----
>   src/mesa/main/pack.c                               | 15 -------
>   src/mesa/main/performance_monitor.c                | 27 -----------
>   src/mesa/state_tracker/st_atom_framebuffer.c       |  8 ++--
>   16 files changed, 10 insertions(+), 201 deletions(-)
>