summaryrefslogtreecommitdiffstats
path: root/video
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2013-11-01 13:00:15 +0100
committerwm4 <wm4@nowhere>2013-11-01 13:00:15 +0100
commitdd344b43e87ef5a24a4ba739821998605e38a015 (patch)
treec0066b9e73fbb6f7ba5a46fed65db9a3a53a9695 /video
parent2696148d64364b40e7b0d1d3de017d6afce4b43d (diff)
downloadmpv-dd344b43e87ef5a24a4ba739821998605e38a015.tar.bz2
mpv-dd344b43e87ef5a24a4ba739821998605e38a015.tar.xz
Enable -Wshadow
This one really did bite me hard (see previous commit), so enable it by default. Fix some cases of shadowing throughout the codebase. None of these change behavior, and all of these were correct code, and just tripped up the warning.
Diffstat (limited to 'video')
-rw-r--r--video/filter/vf_vavpp.c4
-rw-r--r--video/filter/vf_yadif.c12
-rw-r--r--video/out/vo_vdpau.c1
3 files changed, 8 insertions, 9 deletions
diff --git a/video/filter/vf_vavpp.c b/video/filter/vf_vavpp.c
index 62dd9a77a0..f9f77a1b64 100644
--- a/video/filter/vf_vavpp.c
+++ b/video/filter/vf_vavpp.c
@@ -342,8 +342,8 @@ static bool initialize(struct vf_priv_s *p)
if (!num)
continue;
VAProcDeinterlacingType algorithm = VAProcDeinterlacingBob;
- for (int i=0; i<num; ++i) { // find Bob
- if (caps[i].type != algorithm)
+ for (int n=0; n < num; n++) { // find Bob
+ if (caps[n].type != algorithm)
continue;
VAProcFilterParameterBufferDeinterlacing param;
param.type = VAProcFilterDeinterlacing;
diff --git a/video/filter/vf_yadif.c b/video/filter/vf_yadif.c
index b9a48c6da2..b53b1bf594 100644
--- a/video/filter/vf_yadif.c
+++ b/video/filter/vf_yadif.c
@@ -311,16 +311,16 @@ static void filter_line_c(struct vf_priv_s *p, uint8_t *dst, uint8_t *prev, uint
int spatial_score= FFABS(cur[-refs-1] - cur[+refs-1]) + FFABS(c-e)
+ FFABS(cur[-refs+1] - cur[+refs+1]) - 1;
-#define CHECK(j)\
- { int score= FFABS(cur[-refs-1+j] - cur[+refs-1-j])\
+#define CHECK(x, j)\
+ { int score##x= FFABS(cur[-refs-1+j] - cur[+refs-1-j])\
+ FFABS(cur[-refs +j] - cur[+refs -j])\
+ FFABS(cur[-refs+1+j] - cur[+refs+1-j]);\
- if(score < spatial_score){\
- spatial_score= score;\
+ if(score##x < spatial_score){\
+ spatial_score= score##x;\
spatial_pred= (cur[-refs +j] + cur[+refs -j])>>1;\
- CHECK(-1) CHECK(-2) }} }}
- CHECK( 1) CHECK( 2) }} }}
+ CHECK(0, -1) CHECK(1, -2) }} }}
+ CHECK(0, 1) CHECK(1, 2) }} }}
if(p->mode<2){
int b= (prev2[-2*refs] + next2[-2*refs])>>1;
diff --git a/video/out/vo_vdpau.c b/video/out/vo_vdpau.c
index 2140bde948..fd6f87edb9 100644
--- a/video/out/vo_vdpau.c
+++ b/video/out/vo_vdpau.c
@@ -1356,7 +1356,6 @@ static void draw_image(struct vo *vo, mp_image_t *mpi)
return;
surface = (VdpVideoSurface)(intptr_t)reserved_mpi->planes[3];
if (handle_preemption(vo) >= 0) {
- VdpStatus vdp_st;
const void *destdata[3] = {mpi->planes[0], mpi->planes[2],
mpi->planes[1]};
if (vc->image_format == IMGFMT_NV12)