Commit c7bc593d authored by Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf

MKV: code simplification by scope reduction

parent a4cc7556
...@@ -161,7 +161,6 @@ static void MkvTree( demux_t & demuxer, int i_level, const char *psz_format, ... ...@@ -161,7 +161,6 @@ static void MkvTree( demux_t & demuxer, int i_level, const char *psz_format, ...
*****************************************************************************/ *****************************************************************************/
void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
{ {
size_t i, j, k, n;
bool bSupported = true; bool bSupported = true;
mkv_track_t *tk; mkv_track_t *tk;
...@@ -200,7 +199,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -200,7 +199,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
tk->i_compression_type = MATROSKA_COMPRESSION_NONE; tk->i_compression_type = MATROSKA_COMPRESSION_NONE;
tk->p_compression_data = NULL; tk->p_compression_data = NULL;
for( i = 0; i < m->ListSize(); i++ ) for( size_t i = 0; i < m->ListSize(); i++ )
{ {
EbmlElement *l = (*m)[i]; EbmlElement *l = (*m)[i];
...@@ -370,14 +369,14 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -370,14 +369,14 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
msg_Err( &sys.demuxer, "Multiple Compression method not supported" ); msg_Err( &sys.demuxer, "Multiple Compression method not supported" );
bSupported = false; bSupported = false;
} }
for( j = 0; j < cencs->ListSize(); j++ ) for( size_t j = 0; j < cencs->ListSize(); j++ )
{ {
EbmlElement *l2 = (*cencs)[j]; EbmlElement *l2 = (*cencs)[j];
if( MKV_IS_ID( l2, KaxContentEncoding ) ) if( MKV_IS_ID( l2, KaxContentEncoding ) )
{ {
MkvTree( sys.demuxer, 4, "Content Encoding" ); MkvTree( sys.demuxer, 4, "Content Encoding" );
EbmlMaster *cenc = static_cast<EbmlMaster*>(l2); EbmlMaster *cenc = static_cast<EbmlMaster*>(l2);
for( k = 0; k < cenc->ListSize(); k++ ) for( size_t k = 0; k < cenc->ListSize(); k++ )
{ {
EbmlElement *l3 = (*cenc)[k]; EbmlElement *l3 = (*cenc)[k];
if( MKV_IS_ID( l3, KaxContentEncodingOrder ) ) if( MKV_IS_ID( l3, KaxContentEncodingOrder ) )
...@@ -399,7 +398,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -399,7 +398,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
{ {
EbmlMaster *compr = static_cast<EbmlMaster*>(l3); EbmlMaster *compr = static_cast<EbmlMaster*>(l3);
MkvTree( sys.demuxer, 5, "Content Compression" ); MkvTree( sys.demuxer, 5, "Content Compression" );
for( n = 0; n < compr->ListSize(); n++ ) for( size_t n = 0; n < compr->ListSize(); n++ )
{ {
EbmlElement *l4 = (*compr)[n]; EbmlElement *l4 = (*compr)[n];
if( MKV_IS_ID( l4, KaxContentCompAlgo ) ) if( MKV_IS_ID( l4, KaxContentCompAlgo ) )
...@@ -461,7 +460,6 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -461,7 +460,6 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
else if( MKV_IS_ID( l, KaxTrackVideo ) ) else if( MKV_IS_ID( l, KaxTrackVideo ) )
{ {
EbmlMaster *tkv = static_cast<EbmlMaster*>(l); EbmlMaster *tkv = static_cast<EbmlMaster*>(l);
unsigned int j;
unsigned int i_crop_right = 0, i_crop_left = 0, i_crop_top = 0, i_crop_bottom = 0; unsigned int i_crop_right = 0, i_crop_left = 0, i_crop_top = 0, i_crop_bottom = 0;
unsigned int i_display_unit = 0, i_display_width = 0, i_display_height = 0; unsigned int i_display_unit = 0, i_display_width = 0, i_display_height = 0;
...@@ -471,7 +469,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -471,7 +469,7 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
tk->fmt.video.i_frame_rate_base = (unsigned int)(tk->i_default_duration / 1000); tk->fmt.video.i_frame_rate_base = (unsigned int)(tk->i_default_duration / 1000);
tk->fmt.video.i_frame_rate = 1000000; tk->fmt.video.i_frame_rate = 1000000;
for( j = 0; j < tkv->ListSize(); j++ ) for( unsigned int j = 0; j < tkv->ListSize(); j++ )
{ {
EbmlElement *l = (*tkv)[j]; EbmlElement *l = (*tkv)[j];
if( MKV_IS_ID( l, KaxVideoFlagInterlaced ) ) // UNUSED if( MKV_IS_ID( l, KaxVideoFlagInterlaced ) ) // UNUSED
...@@ -595,11 +593,10 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -595,11 +593,10 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
else if( MKV_IS_ID( l, KaxTrackAudio ) ) else if( MKV_IS_ID( l, KaxTrackAudio ) )
{ {
EbmlMaster *tka = static_cast<EbmlMaster*>(l); EbmlMaster *tka = static_cast<EbmlMaster*>(l);
unsigned int j;
msg_Dbg( &sys.demuxer, "| | | + Track Audio" ); msg_Dbg( &sys.demuxer, "| | | + Track Audio" );
for( j = 0; j < tka->ListSize(); j++ ) for( unsigned int j = 0; j < tka->ListSize(); j++ )
{ {
EbmlElement *l = (*tka)[j]; EbmlElement *l = (*tka)[j];
...@@ -661,13 +658,12 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m ) ...@@ -661,13 +658,12 @@ void matroska_segment_c::ParseTrackEntry( KaxTrackEntry *m )
void matroska_segment_c::ParseTracks( KaxTracks *tracks ) void matroska_segment_c::ParseTracks( KaxTracks *tracks )
{ {
EbmlElement *el; EbmlElement *el;
unsigned int i;
int i_upper_level = 0; int i_upper_level = 0;
/* Master elements */ /* Master elements */
tracks->Read( es, tracks->Generic().Context, i_upper_level, el, true ); tracks->Read( es, tracks->Generic().Context, i_upper_level, el, true );
for( i = 0; i < tracks->ListSize(); i++ ) for( size_t i = 0; i < tracks->ListSize(); i++ )
{ {
EbmlElement *l = (*tracks)[i]; EbmlElement *l = (*tracks)[i];
...@@ -689,14 +685,13 @@ void matroska_segment_c::ParseInfo( KaxInfo *info ) ...@@ -689,14 +685,13 @@ void matroska_segment_c::ParseInfo( KaxInfo *info )
{ {
EbmlElement *el; EbmlElement *el;
EbmlMaster *m; EbmlMaster *m;
size_t i, j;
int i_upper_level = 0; int i_upper_level = 0;
/* Master elements */ /* Master elements */
m = static_cast<EbmlMaster *>(info); m = static_cast<EbmlMaster *>(info);
m->Read( es, info->Generic().Context, i_upper_level, el, true ); m->Read( es, info->Generic().Context, i_upper_level, el, true );
for( i = 0; i < m->ListSize(); i++ ) for( size_t i = 0; i < m->ListSize(); i++ )
{ {
EbmlElement *l = (*m)[i]; EbmlElement *l = (*m)[i];
...@@ -804,7 +799,7 @@ void matroska_segment_c::ParseInfo( KaxInfo *info ) ...@@ -804,7 +799,7 @@ void matroska_segment_c::ParseInfo( KaxInfo *info )
chapter_translation_c *p_translate = new chapter_translation_c(); chapter_translation_c *p_translate = new chapter_translation_c();
p_trans->Read( es, p_trans->Generic().Context, i_upper_level, el, true ); p_trans->Read( es, p_trans->Generic().Context, i_upper_level, el, true );
for( j = 0; j < p_trans->ListSize(); j++ ) for( size_t j = 0; j < p_trans->ListSize(); j++ )
{ {
EbmlElement *l = (*p_trans)[j]; EbmlElement *l = (*p_trans)[j];
...@@ -840,10 +835,8 @@ void matroska_segment_c::ParseInfo( KaxInfo *info ) ...@@ -840,10 +835,8 @@ void matroska_segment_c::ParseInfo( KaxInfo *info )
*****************************************************************************/ *****************************************************************************/
void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chapter_item_c & chapters ) void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chapter_item_c & chapters )
{ {
size_t i, j;
msg_Dbg( &sys.demuxer, "| | | + ChapterAtom (level=%d)", i_level ); msg_Dbg( &sys.demuxer, "| | | + ChapterAtom (level=%d)", i_level );
for( i = 0; i < ca->ListSize(); i++ ) for( size_t i = 0; i < ca->ListSize(); i++ )
{ {
EbmlElement *l = (*ca)[i]; EbmlElement *l = (*ca)[i];
...@@ -878,16 +871,14 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap ...@@ -878,16 +871,14 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap
EbmlMaster *cd = static_cast<EbmlMaster *>(l); EbmlMaster *cd = static_cast<EbmlMaster *>(l);
msg_Dbg( &sys.demuxer, "| | | | + ChapterDisplay" ); msg_Dbg( &sys.demuxer, "| | | | + ChapterDisplay" );
for( j = 0; j < cd->ListSize(); j++ ) for( size_t j = 0; j < cd->ListSize(); j++ )
{ {
EbmlElement *l= (*cd)[j]; EbmlElement *l= (*cd)[j];
if( MKV_IS_ID( l, KaxChapterString ) ) if( MKV_IS_ID( l, KaxChapterString ) )
{ {
int k;
KaxChapterString &name =*(KaxChapterString*)l; KaxChapterString &name =*(KaxChapterString*)l;
for (k = 0; k < i_level; k++) for ( int k = 0; k < i_level; k++)
chapters.psz_name += '+'; chapters.psz_name += '+';
chapters.psz_name += ' '; chapters.psz_name += ' ';
char *psz_tmp_utf8 = ToUTF8( UTFstring( name ) ); char *psz_tmp_utf8 = ToUTF8( UTFstring( name ) );
...@@ -920,7 +911,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap ...@@ -920,7 +911,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap
KaxChapterProcess *cp = static_cast<KaxChapterProcess *>(l); KaxChapterProcess *cp = static_cast<KaxChapterProcess *>(l);
chapter_codec_cmds_c *p_ccodec = NULL; chapter_codec_cmds_c *p_ccodec = NULL;
for( j = 0; j < cp->ListSize(); j++ ) for( size_t j = 0; j < cp->ListSize(); j++ )
{ {
EbmlElement *k= (*cp)[j]; EbmlElement *k= (*cp)[j];
...@@ -937,7 +928,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap ...@@ -937,7 +928,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap
if ( p_ccodec != NULL ) if ( p_ccodec != NULL )
{ {
for( j = 0; j < cp->ListSize(); j++ ) for( size_t j = 0; j < cp->ListSize(); j++ )
{ {
EbmlElement *k= (*cp)[j]; EbmlElement *k= (*cp)[j];
...@@ -1012,14 +1003,13 @@ void matroska_segment_c::ParseAttachments( KaxAttachments *attachments ) ...@@ -1012,14 +1003,13 @@ void matroska_segment_c::ParseAttachments( KaxAttachments *attachments )
void matroska_segment_c::ParseChapters( KaxChapters *chapters ) void matroska_segment_c::ParseChapters( KaxChapters *chapters )
{ {
EbmlElement *el; EbmlElement *el;
size_t i;
int i_upper_level = 0; int i_upper_level = 0;
mtime_t i_dur; mtime_t i_dur;
/* Master elements */ /* Master elements */
chapters->Read( es, chapters->Generic().Context, i_upper_level, el, true ); chapters->Read( es, chapters->Generic().Context, i_upper_level, el, true );
for( i = 0; i < chapters->ListSize(); i++ ) for( size_t i = 0; i < chapters->ListSize(); i++ )
{ {
EbmlElement *l = (*chapters)[i]; EbmlElement *l = (*chapters)[i];
...@@ -1069,7 +1059,7 @@ void matroska_segment_c::ParseChapters( KaxChapters *chapters ) ...@@ -1069,7 +1059,7 @@ void matroska_segment_c::ParseChapters( KaxChapters *chapters )
} }
} }
for( i = 0; i < stored_editions.size(); i++ ) for( size_t i = 0; i < stored_editions.size(); i++ )
{ {
stored_editions[i]->RefreshChapters( ); stored_editions[i]->RefreshChapters( );
} }
...@@ -1087,14 +1077,13 @@ void matroska_segment_c::ParseCluster( ) ...@@ -1087,14 +1077,13 @@ void matroska_segment_c::ParseCluster( )
{ {
EbmlElement *el; EbmlElement *el;
EbmlMaster *m; EbmlMaster *m;
unsigned int i;
int i_upper_level = 0; int i_upper_level = 0;
/* Master elements */ /* Master elements */
m = static_cast<EbmlMaster *>( cluster ); m = static_cast<EbmlMaster *>( cluster );
m->Read( es, cluster->Generic().Context, i_upper_level, el, true ); m->Read( es, cluster->Generic().Context, i_upper_level, el, true );
for( i = 0; i < m->ListSize(); i++ ) for( unsigned int i = 0; i < m->ListSize(); i++ )
{ {
EbmlElement *l = (*m)[i]; EbmlElement *l = (*m)[i];
...@@ -1110,4 +1099,3 @@ void matroska_segment_c::ParseCluster( ) ...@@ -1110,4 +1099,3 @@ void matroska_segment_c::ParseCluster( )
i_start_time = cluster->GlobalTimecode() / 1000; i_start_time = cluster->GlobalTimecode() / 1000;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment