[Mesa-dev] [PATCH] draw: Remove unused variables.
Dave Airlie
airlied at gmail.com
Sat Jan 28 10:33:46 PST 2012
On Sat, Jan 28, 2012 at 6:03 PM, Jose Fonseca <jfonseca at vmware.com> wrote:
> Regardless of Dave's intentions there's no harm in removing it.
Keeps falling off my radar this patch,
yeah I don't think it was required what I was going to do, and I have
to fix up interpolation stuff a bit more later anyways.
Reviewed-by: Dave Airlie <airlied at redhat.com>
>
> Jose
>
> ----- Original Message -----
>> Dave. Ping.
>>
>> On Mon, Jan 16, 2012 at 3:46 AM, Jose Fonseca <jfonseca at vmware.com>
>> wrote:
>> > Hi Vinson,
>> >
>> > These lines were added by Dave in
>> >
>> > commit 1865f341d8f45b389061fc08d2da90b7aa8a6099
>> > Author: Dave Airlie <airlied at redhat.com>
>> > Date: Fri Jan 6 12:23:00 2012 +0000
>> >
>> > draw: clipdistance support (v2)
>> >
>> > Add support for using the clipdistance instead of clip plane.
>> >
>> > Passes all piglit clipdistance tests.
>> >
>> > v2: fixup some comments from Brian in review.
>> >
>> > Signed-off-by: Dave Airlie <airlied at redhat.com>
>> >
>> >
>> > So Dave should take a look, and determine whether this is really
>> > dead code, or code is missing.
>> >
>> > Jose
>> >
>> >
>> >
>> > ----- Original Message -----
>> >> Fix this GCC warning.
>> >> draw_pipe_clip.c: In function ‘interp’:
>> >> draw_pipe_clip.c:122:13: warning: variable ‘clip_dist’ set but not
>> >> used
>> >> [-Wunused-but-set-variable]
>> >>
>> >> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
>> >> ---
>> >> src/gallium/auxiliary/draw/draw_pipe_clip.c | 4 ----
>> >> 1 files changed, 0 insertions(+), 4 deletions(-)
>> >>
>> >> diff --git a/src/gallium/auxiliary/draw/draw_pipe_clip.c
>> >> b/src/gallium/auxiliary/draw/draw_pipe_clip.c
>> >> index f701972..4da4d65 100644
>> >> --- a/src/gallium/auxiliary/draw/draw_pipe_clip.c
>> >> +++ b/src/gallium/auxiliary/draw/draw_pipe_clip.c
>> >> @@ -119,12 +119,8 @@ static void interp( const struct clip_stage
>> >> *clip,
>> >> const unsigned nr_attrs =
>> >> draw_current_shader_outputs(clip->stage.draw);
>> >> const unsigned pos_attr =
>> >> draw_current_shader_position_output(clip->stage.draw);
>> >> const unsigned clip_attr =
>> >> draw_current_shader_clipvertex_output(clip->stage.draw);
>> >> - unsigned clip_dist[2];
>> >> unsigned j;
>> >>
>> >> - clip_dist[0] =
>> >> draw_current_shader_clipdistance_output(clip->stage.draw, 0);
>> >> - clip_dist[1] =
>> >> draw_current_shader_clipdistance_output(clip->stage.draw, 1);
>> >> -
>> >> /* Vertex header.
>> >> */
>> >> dst->clipmask = 0;
>> >> --
>> >> 1.7.8.3
>> >>
>> >> _______________________________________________
>> >> mesa-dev mailing list
>> >> mesa-dev at lists.freedesktop.org
>> >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>> >>
>>
More information about the mesa-dev
mailing list