X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fvideo_filter%2Frotate.c;h=52bc735ad08cdec172d2cd72245a7ae8321d267a;hb=6c29b00af01a5570e49b2c56e36650882903a14f;hp=8a1d4342aacaea22845550781a48ee3e03c04c63;hpb=74e5a0727b83ea8bd5ee87dbce7596b990ef14c1;p=vlc diff --git a/modules/video_filter/rotate.c b/modules/video_filter/rotate.c index 8a1d4342aa..52bc735ad0 100644 --- a/modules/video_filter/rotate.c +++ b/modules/video_filter/rotate.c @@ -34,7 +34,7 @@ #include #include -#include "vlc_filter.h" +#include #include "filter_picture.h" /***************************************************************************** @@ -122,7 +122,7 @@ static int Create( vlc_object_t *p_this ) break; default: - msg_Err( p_filter, "Unsupported input chroma (%4s)", + msg_Err( p_filter, "Unsupported input chroma (%4.4s)", (char*)&(p_filter->fmt_in.video.i_chroma) ); return VLC_EGENERIC; } @@ -146,7 +146,6 @@ static int Create( vlc_object_t *p_this ) var_AddCallback( p_filter, FILTER_PREFIX "deciangle", PreciseRotateCallback, p_sys ); - return VLC_SUCCESS; } @@ -171,7 +170,6 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) { picture_t *p_outpic; filter_sys_t *p_sys = p_filter->p_sys; - int i_sin, i_cos; if( !p_pic ) return NULL; @@ -183,28 +181,24 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) } vlc_spin_lock( &p_sys->lock ); - i_sin = p_sys->i_sin; - i_cos = p_sys->i_cos; + const int i_sin = p_sys->i_sin; + const int i_cos = p_sys->i_cos; vlc_spin_unlock( &p_sys->lock ); for( int i_plane = 0 ; i_plane < p_pic->i_planes ; i_plane++ ) { - const int i_visible_lines = p_pic->p[i_plane].i_visible_lines; - const int i_visible_pitch = p_pic->p[i_plane].i_visible_pitch; - const int i_pitch = p_pic->p[i_plane].i_pitch; - const int i_hidden_pitch = i_pitch - i_visible_pitch; + plane_t *p_srcp = &p_pic->p[i_plane]; + plane_t *p_dstp = &p_outpic->p[i_plane]; - const int i_aspect = ( i_visible_lines * p_pic->p[Y_PLANE].i_visible_pitch ) / ( p_pic->p[Y_PLANE].i_visible_lines * i_visible_pitch ); + const int i_visible_lines = p_srcp->i_visible_lines; + const int i_visible_pitch = p_srcp->i_visible_pitch; + + const int i_aspect = __MAX( 1, ( i_visible_lines * p_pic->p[Y_PLANE].i_visible_pitch ) / ( p_pic->p[Y_PLANE].i_visible_lines * i_visible_pitch )); /* = 2 for U and V planes in YUV 4:2:2, = 1 otherwise */ const int i_line_center = i_visible_lines>>1; const int i_col_center = i_visible_pitch>>1; - const uint8_t *p_in = p_pic->p[i_plane].p_pixels; - uint8_t *p_out = p_outpic->p[i_plane].p_pixels; - uint8_t *p_outendline = p_out + i_visible_pitch; - const uint8_t *p_outend = p_out + i_visible_lines * i_pitch; - const uint8_t black_pixel = ( i_plane == Y_PLANE ) ? 0x00 : 0x80; const int i_line_next = i_cos / i_aspect -i_sin*i_visible_pitch; @@ -213,17 +207,15 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) - i_sin * i_col_center + (1<<11) ); int i_col_orig0 = i_sin * i_line_center / i_aspect - i_cos * i_col_center + (1<<11); - for( ; p_outendline < p_outend; - p_out += i_hidden_pitch, p_outendline += i_pitch, - i_line_orig0 += i_line_next, i_col_orig0 += i_col_next ) + for( int y = 0; y < i_visible_lines; y++) { - for( ; p_out < p_outendline; - p_out++, i_line_orig0 += i_sin, i_col_orig0 += i_cos ) + uint8_t *p_out = &p_dstp->p_pixels[y * p_dstp->i_pitch]; + + for( int x = 0; x < i_visible_pitch; x++, p_out++ ) { const int i_line_orig = (i_line_orig0>>12)*i_aspect + i_line_center; const int i_col_orig = (i_col_orig0>>12) + i_col_center; - const uint8_t* p_orig_offset = p_in + i_line_orig * i_pitch - + i_col_orig; + const uint8_t *p_orig_offset = &p_srcp->p_pixels[i_line_orig * p_srcp->i_pitch + i_col_orig]; const uint8_t i_line_percent = (i_line_orig0>>4) & 255; const uint8_t i_col_percent = (i_col_orig0 >>4) & 255; @@ -248,7 +240,7 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) && ( i_line_orig >= 0 ) ) i_colpix = *p_orig_offset; - p_orig_offset+=i_pitch; + p_orig_offset += p_srcp->i_pitch; if( ( i_line_orig < i_visible_lines - 1) && ( i_col_orig < i_visible_pitch - 1) ) i_nexpix = *p_orig_offset; @@ -282,7 +274,13 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) { *p_out = black_pixel; } + + i_line_orig0 += i_sin; + i_col_orig0 += i_cos; } + + i_line_orig0 += i_line_next; + i_col_orig0 += i_col_next; } } @@ -296,7 +294,6 @@ static picture_t *FilterPacked( filter_t *p_filter, picture_t *p_pic ) { picture_t *p_outpic; filter_sys_t *p_sys = p_filter->p_sys; - const int i_sin = p_sys->i_sin, i_cos = p_sys->i_cos; if( !p_pic ) return NULL; @@ -305,7 +302,7 @@ static picture_t *FilterPacked( filter_t *p_filter, picture_t *p_pic ) if( GetPackedYuvOffsets( p_pic->format.i_chroma, &i_y_offset, &i_u_offset, &i_v_offset ) != VLC_SUCCESS ) { - msg_Warn( p_filter, "Unsupported input chroma (%4s)", + msg_Warn( p_filter, "Unsupported input chroma (%4.4s)", (char*)&(p_pic->format.i_chroma) ); picture_Release( p_pic ); return NULL; @@ -318,21 +315,27 @@ static picture_t *FilterPacked( filter_t *p_filter, picture_t *p_pic ) return NULL; } + const int i_visible_pitch = p_pic->p->i_visible_pitch>>1; /* In fact it's i_visible_pixels */ + const int i_visible_lines = p_pic->p->i_visible_lines; + const uint8_t *p_in = p_pic->p->p_pixels+i_y_offset; const uint8_t *p_in_u = p_pic->p->p_pixels+i_u_offset; const uint8_t *p_in_v = p_pic->p->p_pixels+i_v_offset; - - const int i_pitch = p_pic->p->i_pitch; - const int i_visible_pitch = p_pic->p->i_visible_pitch>>1; /* In fact it's i_visible_pixels */ - const int i_visible_lines = p_pic->p->i_visible_lines; + const int i_in_pitch = p_pic->p->i_pitch; uint8_t *p_out = p_outpic->p->p_pixels+i_y_offset; uint8_t *p_out_u = p_outpic->p->p_pixels+i_u_offset; uint8_t *p_out_v = p_outpic->p->p_pixels+i_v_offset; + const int i_out_pitch = p_outpic->p->i_pitch; const int i_line_center = i_visible_lines>>1; const int i_col_center = i_visible_pitch>>1; + vlc_spin_lock( &p_sys->lock ); + const int i_sin = p_sys->i_sin; + const int i_cos = p_sys->i_cos; + vlc_spin_unlock( &p_sys->lock ); + int i_col, i_line; for( i_line = 0; i_line < i_visible_lines; i_line++ ) { @@ -350,16 +353,16 @@ static picture_t *FilterPacked( filter_t *p_filter, picture_t *p_pic ) if( 0 <= i_col_orig && i_col_orig < i_visible_pitch && 0 <= i_line_orig && i_line_orig < i_visible_lines ) { - p_out[i_line*i_pitch+2*i_col] = p_in[i_line_orig*i_pitch+2*i_col_orig]; + p_out[i_line*i_out_pitch+2*i_col] = p_in[i_line_orig*i_in_pitch+2*i_col_orig]; i_col_orig /= 2; - p_out_u[i_line*i_pitch+2*i_col] = p_in_u[i_line_orig*i_pitch+4*i_col_orig]; - p_out_v[i_line*i_pitch+2*i_col] = p_in_v[i_line_orig*i_pitch+4*i_col_orig]; + p_out_u[i_line*i_out_pitch+2*i_col] = p_in_u[i_line_orig*i_in_pitch+4*i_col_orig]; + p_out_v[i_line*i_out_pitch+2*i_col] = p_in_v[i_line_orig*i_in_pitch+4*i_col_orig]; } else { - p_out[i_line*i_pitch+2*i_col] = 0x00; - p_out_u[i_line*i_pitch+2*i_col] = 0x80; - p_out_v[i_line*i_pitch+2*i_col] = 0x80; + p_out[i_line*i_out_pitch+2*i_col] = 0x00; + p_out_u[i_line*i_out_pitch+2*i_col] = 0x80; + p_out_v[i_line*i_out_pitch+2*i_col] = 0x80; } /* Handle "2nd Y" */ @@ -376,11 +379,11 @@ static picture_t *FilterPacked( filter_t *p_filter, picture_t *p_pic ) if( 0 <= i_col_orig && i_col_orig < i_visible_pitch && 0 <= i_line_orig && i_line_orig < i_visible_lines ) { - p_out[i_line*i_pitch+2*i_col] = p_in[i_line_orig*i_pitch+2*i_col_orig]; + p_out[i_line*i_out_pitch+2*i_col] = p_in[i_line_orig*i_in_pitch+2*i_col_orig]; } else { - p_out[i_line*i_pitch+2*i_col] = 0x00; + p_out[i_line*i_out_pitch+2*i_col] = 0x00; } } }