i_posy = val.i_int;
/* Check if subpicture position was overridden */
- p_sys->b_absolute = VLC_TRUE;
+ p_sys->b_absolute = VLC_FALSE;
p_sys->i_spu_x = p_sys->i_spu_y = 0;
- if( i_posx >= 0 && i_posy >= 0 )
+ if( ( i_posx >= 0 ) && ( i_posy >= 0 ) )
{
- p_sys->b_absolute = VLC_FALSE;
+ p_sys->b_absolute = VLC_TRUE;
p_sys->i_spu_x = i_posx;
p_sys->i_spu_y = i_posy;
}
block_t *p_block;
subpicture_t *p_spu = NULL;
- if( pp_block == NULL || *pp_block == NULL ) return NULL;
+ if( ( pp_block == NULL ) || ( *pp_block == NULL ) ) return NULL;
p_block = *pp_block;
*pp_block = NULL;
}
/* Check if the page is to be displayed */
- if( p_sys->p_page && p_sys->b_page ) p_spu = render( p_dec );
+ if( p_sys->p_page && p_sys->b_page )
+ p_spu = render( p_dec );
block_Release( p_block );
/* segment size */
i_size = bs_show( s, 16 );
- if( i_page_id != p_sys->i_id && i_page_id != p_sys->i_ancillary_id )
+ if( ( i_page_id != p_sys->i_id ) &&
+ ( i_page_id != p_sys->i_ancillary_id ) )
{
#ifdef DEBUG_DVBSUB
msg_Dbg( p_dec, "subtitle skipped (page id: %i, %i)",
return;
}
- if( p_sys->i_ancillary_id != p_sys->i_id &&
- i_type == DVBSUB_ST_PAGE_COMPOSITION &&
- i_page_id == p_sys->i_ancillary_id )
+ if( ( p_sys->i_ancillary_id != p_sys->i_id ) &&
+ ( i_type == DVBSUB_ST_PAGE_COMPOSITION ) &&
+ ( i_page_id == p_sys->i_ancillary_id ) )
{
#ifdef DEBUG_DVBSUB
msg_Dbg( p_dec, "skipped invalid ancillary subtitle packet" );
}
/* Check version number */
- if( p_clut && p_clut->i_version == i_version )
+ if( p_clut && ( p_clut->i_version == i_version ) )
{
/* Nothing to do */
bs_skip( s, 8 * i_segment_length - 12 );
/* According to EN 300-743 section 7.2.3 note 1, type should
* not have more than 1 bit set to one, but some streams don't
* respect this note. */
- if( i_type & 0x04 && i_id < 4 )
+ if( ( i_type & 0x04 ) && ( i_id < 4 ) )
{
p_clut->c_2b[i_id].Y = y;
p_clut->c_2b[i_id].Cr = cr;
p_clut->c_2b[i_id].Cb = cb;
p_clut->c_2b[i_id].T = t;
}
- if( i_type & 0x02 && i_id < 16 )
+ if( ( i_type & 0x02 ) && ( i_id < 16 ) )
{
p_clut->c_4b[i_id].Y = y;
p_clut->c_4b[i_id].Cr = cr;
#endif
free_all( p_dec );
}
- else if( !p_sys->p_page && i_state != DVBSUB_PCS_STATE_ACQUISITION &&
- i_state != DVBSUB_PCS_STATE_CHANGE )
+ else if( !p_sys->p_page && ( i_state != DVBSUB_PCS_STATE_ACQUISITION ) &&
+ ( i_state != DVBSUB_PCS_STATE_CHANGE ) )
{
/* Not a full PCS, we need to wait for one */
msg_Dbg( p_dec, "didn't receive an acquisition page yet" );
#endif
/* Check version number */
- if( p_sys->p_page && p_sys->p_page->i_version == i_version )
+ if( p_sys->p_page && ( p_sys->p_page->i_version == i_version ) )
{
bs_skip( s, 8 * (i_segment_length - 2) );
return;
{
if( p_sys->p_page->i_region_defs )
free( p_sys->p_page->p_region_defs );
+ p_sys->p_page->p_region_defs = NULL;
p_sys->p_page->i_region_defs = 0;
}
p_sys->p_page->p_region_defs =
malloc( p_sys->p_page->i_region_defs * sizeof(dvbsub_region_t) );
- for( i = 0; i < p_sys->p_page->i_region_defs; i++ )
+ if( p_sys->p_page->p_region_defs )
{
- p_sys->p_page->p_region_defs[i].i_id = bs_read( s, 8 );
- bs_skip( s, 8 ); /* Reserved */
- p_sys->p_page->p_region_defs[i].i_x = bs_read( s, 16 );
- p_sys->p_page->p_region_defs[i].i_y = bs_read( s, 16 );
+ for( i = 0; i < p_sys->p_page->i_region_defs; i++ )
+ {
+ p_sys->p_page->p_region_defs[i].i_id = bs_read( s, 8 );
+ bs_skip( s, 8 ); /* Reserved */
+ p_sys->p_page->p_region_defs[i].i_x = bs_read( s, 16 );
+ p_sys->p_page->p_region_defs[i].i_y = bs_read( s, 16 );
#ifdef DEBUG_DVBSUB
- msg_Dbg( p_dec, "page_composition, region %i (%i,%i)",
- i, p_sys->p_page->p_region_defs[i].i_x,
- p_sys->p_page->p_region_defs[i].i_y );
+ msg_Dbg( p_dec, "page_composition, region %i (%i,%i)",
+ i, p_sys->p_page->p_region_defs[i].i_x,
+ p_sys->p_page->p_region_defs[i].i_y );
#endif
+ }
}
}
}
/* Check version number */
- if( p_region && p_region->i_version == i_version )
+ if( p_region && ( p_region->i_version == i_version ) )
{
bs_skip( s, 8 * (i_segment_length - 1) - 4 );
return;
msg_Dbg( p_dec, "new region: %i", i_id );
#endif
p_region = *pp_region = malloc( sizeof(dvbsub_region_t) );
- memset( p_region, 0, sizeof(dvbsub_region_t) );
- p_region->p_object_defs = NULL;
- p_region->p_pixbuf = NULL;
- p_region->p_next = NULL;
+ if( p_region )
+ {
+ memset( p_region, 0, sizeof(dvbsub_region_t) );
+ p_region->p_object_defs = NULL;
+ p_region->p_pixbuf = NULL;
+ p_region->p_next = NULL;
+ }
}
/* Region attributes */
int i = p_region->i_object_defs - 1;
if( p_region->p_object_defs[i].psz_text )
free( p_region->p_object_defs[i].psz_text );
- if( !i ) free( p_region->p_object_defs );
+ if( !i )
+ free( p_region->p_object_defs );
p_region->i_object_defs--;
}
p_region->p_object_defs = NULL;
/* Extra sanity checks */
- if( p_region->i_width != i_width || p_region->i_height != i_height )
+ if( ( p_region->i_width != i_width ) ||
+ ( p_region->i_height != i_height ) )
{
if( p_region->p_pixbuf )
{
p_region->i_depth = 0;
b_fill = VLC_TRUE;
}
- if( p_region->i_depth && (p_region->i_depth != i_depth ||
- p_region->i_level_comp != i_level_comp || p_region->i_clut != i_clut) )
+ if( p_region->i_depth &&
+ ( ( p_region->i_depth != i_depth ) ||
+ ( p_region->i_level_comp != i_level_comp ) ||
+ ( p_region->i_clut != i_clut) ) )
{
msg_Dbg( p_dec, "region parameters changed (not allowed)" );
}
/* Erase background of region */
if( b_fill )
{
- int i_background = (p_region->i_depth == 1) ? i_2_bg :
- (p_region->i_depth == 2) ? i_4_bg : i_8_bg;
+ int i_background = ( p_region->i_depth == 1 ) ? i_2_bg :
+ ( ( p_region->i_depth == 2 ) ? i_4_bg : i_8_bg );
memset( p_region->p_pixbuf, i_background, i_width * i_height );
}
i_processed_length += 6;
- if( p_obj->i_type == DVBSUB_OT_BASIC_CHAR ||
- p_obj->i_type == DVBSUB_OT_COMPOSITE_STRING )
+ if( ( p_obj->i_type == DVBSUB_OT_BASIC_CHAR ) ||
+ ( p_obj->i_type == DVBSUB_OT_COMPOSITE_STRING ) )
{
p_obj->i_fg_pc = bs_read( s, 8 );
p_obj->i_bg_pc = bs_read( s, 8 );
i_version = bs_read( s, 4 );
/* Check version number */
- if( p_old && p_old->i_version == i_version )
+ if( p_old && ( p_old->i_version == i_version ) )
{
/* The definition did not change */
bs_skip( s, 8*i_segment_length - 4 );
msg_Dbg( p_dec, "new display definition: %i", i_version );
#endif
p_display = malloc( sizeof(dvbsub_display_t) );
-
- /* We don't have this version of the display definition: Parse it */
- p_display->i_version = i_version;
- p_display->b_windowed = bs_read( s, 1 );
- bs_skip( s, 3 ); /* Reserved bits */
- p_display->i_width = bs_read( s, 16 )+1;
- p_display->i_height = bs_read( s, 16 )+1;
-
- if( p_display->b_windowed )
+ if( p_display )
{
+ /* We don't have this version of the display definition: Parse it */
+ p_display->i_version = i_version;
+ p_display->b_windowed = bs_read( s, 1 );
+ bs_skip( s, 3 ); /* Reserved bits */
+ p_display->i_width = bs_read( s, 16 )+1;
+ p_display->i_height = bs_read( s, 16 )+1;
+
+ if( p_display->b_windowed )
+ {
#ifdef DEBUG_DVBSUB
- msg_Dbg( p_dec, "display definition with offsets (windowed)" );
+ msg_Dbg( p_dec, "display definition with offsets (windowed)" );
#endif
- /* Coordinates are measured from the top left corner */
- p_display->i_x = bs_read( s, 16 );
- p_display->i_max_x = bs_read( s, 16 );
- p_display->i_y = bs_read( s, 16 );
- p_display->i_max_y = bs_read( s, 16 );
- i_processed_length += 64;
+ /* Coordinates are measured from the top left corner */
+ p_display->i_x = bs_read( s, 16 );
+ p_display->i_max_x = bs_read( s, 16 );
+ p_display->i_y = bs_read( s, 16 );
+ p_display->i_max_y = bs_read( s, 16 );
+ i_processed_length += 64;
+ }
}
p_sys->p_display = p_display;
if( i_processed_length != i_segment_length*8 )
{
- msg_Err( p_dec, "processed length %d != segment length %d", i_processed_length, i_segment_length );
+ msg_Err( p_dec, "processed length %d != segment length %d",
+ i_processed_length, i_segment_length );
}
#ifdef DEBUG_DVBSUB
- msg_Dbg( p_dec, "version: %d, width: %d, height: %d", p_display->i_version, p_display->i_width, p_display->i_height );
+ msg_Dbg( p_dec, "version: %d, width: %d, height: %d",
+ p_display->i_version, p_display->i_width, p_display->i_height );
if( p_display->b_windowed )
- msg_Dbg( p_dec, "xmin: %d, xmax: %d, ymin: %d, ymax: %d", p_display->i_x, p_display->i_max_x, p_display->i_y, p_display->i_max_y );
+ msg_Dbg( p_dec, "xmin: %d, xmax: %d, ymin: %d, ymax: %d",
+ p_display->i_x, p_display->i_max_x, p_display->i_y, p_display->i_max_y );
#endif
}
bs_skip( s, 8 * (i_segment_length - 7) );
/* Sanity check */
- if( i_segment_length < i_topfield + i_bottomfield + 7 ||
- p_topfield + i_topfield + i_bottomfield > s->p_end )
+ if( ( i_segment_length < ( i_topfield + i_bottomfield + 7 ) ) ||
+ ( ( p_topfield + i_topfield + i_bottomfield ) > s->p_end ) )
{
msg_Dbg( p_dec, "corrupted object data" );
return;
uint8_t* p_start = s->p_start + bs_pos( s ) / 8;
/* Sanity check */
- if( i_segment_length < i_number_of_codes*2 + 4 ||
- p_start + i_number_of_codes*2 > s->p_end )
+ if( ( i_segment_length < ( i_number_of_codes*2 + 4 ) ) ||
+ ( ( p_start + i_number_of_codes*2 ) > s->p_end ) )
{
msg_Dbg( p_dec, "corrupted object data" );
return;
{
p_region->p_object_defs[i].psz_text[j] = (char)(bs_read( s, 16 ) & 0xFF);
}
+ /* Null terminate the string */
p_region->p_object_defs[i].psz_text[j] = 0;
}
}
static void dvbsub_pdata2bpp( bs_t *s, uint8_t *p, int i_width, int *pi_off )
{
- vlc_bool_t b_stop = 0;
+ vlc_bool_t b_stop = VLC_FALSE;
while( !b_stop && !bs_eof( s ) )
{
int i_count = 0, i_color = 0;
- if( (i_color = bs_read( s, 2 )) != 0x00 )
+ i_color = bs_read( s, 2 );
+ if( i_color != 0x00 )
{
i_count = 1;
}
switch( bs_read( s, 2 ) ) //Switch3
{
case 0x00:
- b_stop = 1;
+ b_stop = VLC_TRUE;
break;
case 0x01:
i_count = 2;
if( !i_count ) continue;
/* Sanity check */
- if( i_count + *pi_off > i_width ) break;
+ if( ( i_count + *pi_off ) > i_width ) break;
if( i_count == 1 ) p[*pi_off] = i_color;
- else memset( p + *pi_off, i_color, i_count );
+ else memset( ( p + *pi_off ), i_color, i_count );
(*pi_off) += i_count;
}
static void dvbsub_pdata4bpp( bs_t *s, uint8_t *p, int i_width, int *pi_off )
{
- vlc_bool_t b_stop = 0;
+ vlc_bool_t b_stop = VLC_FALSE;
while( !b_stop && !bs_eof( s ) )
{
int i_count = 0, i_color = 0;
- if( (i_color = bs_read( s, 4 )) != 0x00 )
+ i_color = bs_read( s, 4 );
+ if( i_color != 0x00 )
{
/* Add 1 pixel */
i_count = 1;
else
{
bs_skip( s, 3 );
- b_stop = 1;
+ b_stop = VLC_TRUE;
}
}
else
if( !i_count ) continue;
/* Sanity check */
- if( i_count + *pi_off > i_width ) break;
+ if( ( i_count + *pi_off ) > i_width ) break;
if( i_count == 1 ) p[*pi_off] = i_color;
- else memset( p + *pi_off, i_color, i_count );
+ else memset( ( p + *pi_off ), i_color, i_count );
(*pi_off) += i_count;
}
static void dvbsub_pdata8bpp( bs_t *s, uint8_t *p, int i_width, int *pi_off )
{
- vlc_bool_t b_stop = 0;
+ vlc_bool_t b_stop = VLC_FALSE;
while( !b_stop && !bs_eof( s ) )
{
int i_count = 0, i_color = 0;
- if( (i_color = bs_read( s, 8 )) != 0x00 )
+ i_color = bs_read( s, 8 );
+ if( i_color != 0x00 )
{
/* Add 1 pixel */
i_count = 1;
else
{
bs_skip( s, 7 );
- b_stop = 1;
+ b_stop = VLC_TRUE;
}
}
else
if( !i_count ) continue;
/* Sanity check */
- if( i_count + *pi_off > i_width ) break;
+ if( ( i_count + *pi_off ) > i_width ) break;
if( i_count == 1 ) p[*pi_off] = i_color;
- else memset( p + *pi_off, i_color, i_count );
+ else memset( ( p + *pi_off ), i_color, i_count );
(*pi_off) += i_count;
}
msg_Dbg( p_dec, "rendering %i regions", p_sys->p_page->i_region_defs );
#endif
- for( i = 0; p_sys->p_page && i < p_sys->p_page->i_region_defs; i++ )
+ for( i = 0; p_sys->p_page && ( i < p_sys->p_page->i_region_defs ); i++ )
{
dvbsub_region_t *p_region;
dvbsub_regiondef_t *p_regiondef;
continue;
}
+ /* FIXME: don't create a subpicture region with VLC_FOURCC YUVP
+ * when it actually is a TEXT region */
+
/* Create new SPU region */
memset( &fmt, 0, sizeof(video_format_t) );
fmt.i_chroma = VLC_FOURCC('Y','U','V','P');
pp_spu_region = &p_spu_region->p_next;
/* Build palette */
- fmt.p_palette->i_entries = p_region->i_depth == 1 ? 4 :
- p_region->i_depth == 2 ? 16 : 256;
- p_color = (p_region->i_depth == 1) ? p_clut->c_2b :
- (p_region->i_depth == 2) ? p_clut->c_4b : p_clut->c_8b;
+ fmt.p_palette->i_entries = ( p_region->i_depth == 1 ) ? 4 :
+ ( ( p_region->i_depth == 2 ) ? 16 : 256 );
+ p_color = ( p_region->i_depth == 1 ) ? p_clut->c_2b :
+ ( ( p_region->i_depth == 2 ) ? p_clut->c_4b : p_clut->c_8b );
for( j = 0; j < fmt.p_palette->i_entries; j++ )
{
fmt.p_palette->palette[j][0] = p_color[j].Y;
{
dvbsub_objectdef_t *p_object_def = &p_region->p_object_defs[j];
- if( p_object_def->i_type != 1 || !p_object_def->psz_text )
+ if( ( p_object_def->i_type != 1 ) || !p_object_def->psz_text )
continue;
/* Create new SPU region */
}
/* Set the pf_render callback */
- p_spu->i_start = p_sys->i_pts;
+ p_spu->i_start = (mtime_t) p_sys->i_pts;
+ //p_spu->i_stop = (mtime_t) 0;
p_spu->b_ephemer = VLC_TRUE;
p_spu->b_pausable = VLC_TRUE;
//p_spu->b_fade = VLC_TRUE;
encoder_sys_t *p_sys;
vlc_value_t val;
- if( p_enc->fmt_out.i_codec != VLC_FOURCC('d','v','b','s') &&
+ if( ( p_enc->fmt_out.i_codec != VLC_FOURCC('d','v','b','s') ) &&
!p_enc->b_force )
{
return VLC_EGENERIC;
/* Sanity check */
if( !p_region ) return NULL;
- if( p_region->fmt.i_chroma != VLC_FOURCC('T','E','X','T') &&
- p_region->fmt.i_chroma != VLC_FOURCC('Y','U','V','P') )
+ if( ( p_region->fmt.i_chroma != VLC_FOURCC('T','E','X','T') ) &&
+ ( p_region->fmt.i_chroma != VLC_FOURCC('Y','U','V','P') ) )
{
char psz_fourcc[5];
memset( &psz_fourcc, 0, sizeof( psz_fourcc ) );
bs_write( s, 8, 0xff );/* End marker */
p_block->i_buffer = bs_pos( s ) / 8;
p_block->i_pts = p_block->i_dts = p_subpic->i_start;
- if( !p_subpic->b_ephemer && p_subpic->i_stop > p_subpic->i_start )
+ if( !p_subpic->b_ephemer && ( p_subpic->i_stop > p_subpic->i_start ) )
{
block_t *p_block_stop;
i_timeout = 0;
if( p_subpic && !p_subpic->b_ephemer &&
- p_subpic->i_stop > p_subpic->i_start )
+ ( p_subpic->i_stop > p_subpic->i_start ) )
{
i_timeout = (p_subpic->i_stop - p_subpic->i_start) / 1000000;
}
{
int i_entries = 4, i_depth = 0x1, i_bg = 0;
vlc_bool_t b_text =
- p_region->fmt.i_chroma == VLC_FOURCC('T','E','X','T');
+ ( p_region->fmt.i_chroma == VLC_FOURCC('T','E','X','T') );
if( !b_text )
{
for( i = 0; i <= p_region->fmt.i_visible_width; i++ )
{
- if( i != p_region->fmt.i_visible_width &&
- p_data[i] == i_last_pixel && i_length != 284 )
+ if( ( i != p_region->fmt.i_visible_width ) &&
+ ( p_data[i] == i_last_pixel ) && ( i_length != 284 ) )
{
i_length++;
continue;
}
- if( i_length == 1 || i_length == 11 || i_length == 28 )
+ if( ( i_length == 1 ) || ( i_length == 11 ) || ( i_length == 28 ) )
{
/* 2bit/pixel code */
- if( i_last_pixel ) bs_write( s, 2, i_last_pixel );
+ if( i_last_pixel )
+ bs_write( s, 2, i_last_pixel );
else
{
bs_write( s, 2, 0 );
continue;
}
- if( i_length == 1 || (i_length == 3 && i_last_pixel) || i_length == 8 )
+ if( ( i_length == 1 ) ||
+ ( ( i_length == 3 ) && i_last_pixel ) ||
+ ( i_length == 8 ) )
{
/* 4bit/pixel code */
- if( i_last_pixel ) bs_write( s, 4, i_last_pixel );
+ if( i_last_pixel )
+ bs_write( s, 4, i_last_pixel );
else
{
bs_write( s, 4, 0 );
bs_write( s, 2, 1 ); /* 2 * pseudo color 0 */
}
}
- else if( !i_last_pixel && i_length >= 3 && i_length <= 9 )
+ else if( !i_last_pixel && ( i_length >= 3 ) && ( i_length <= 9 ) )
{
bs_write( s, 4, 0 );
bs_write( s, 1, 0 );
for( i = 0; i <= p_region->fmt.i_visible_width; i++ )
{
- if( i != p_region->fmt.i_visible_width &&
- p_data[i] == i_last_pixel && i_length != 127 )
+ if( ( i != p_region->fmt.i_visible_width ) &&
+ ( p_data[i] == i_last_pixel ) && ( i_length != 127 ) )
{
i_length++;
continue;
}
- if( i_length == 1 && i_last_pixel )
+ if( ( i_length == 1 ) && i_last_pixel )
{
/* 8bit/pixel code */
bs_write( s, 8, i_last_pixel );
}
- else if( i_length == 2 && i_last_pixel )
+ else if( ( i_length == 2 ) && i_last_pixel )
{
/* 8bit/pixel code */
bs_write( s, 8, i_last_pixel );