[vlc-commits] demux:mkv: remove useless wrapping
Steve Lhomme
git at videolan.org
Fri Jan 19 11:54:53 CET 2018
vlc | branch: master | Steve Lhomme <robux4 at ycbcr.xyz> | Fri Jan 19 11:32:09 2018 +0100| [bb5fef95831464aec53b8ab45338d8cbd593fb42] | committer: Steve Lhomme
demux:mkv: remove useless wrapping
It's converted to void* anyway
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=bb5fef95831464aec53b8ab45338d8cbd593fb42
---
modules/demux/mkv/dispatcher.hpp | 3 ---
modules/demux/mkv/matroska_segment.cpp | 12 ++++-----
modules/demux/mkv/matroska_segment_parse.cpp | 37 +++++++++++++---------------
3 files changed, 23 insertions(+), 29 deletions(-)
diff --git a/modules/demux/mkv/dispatcher.hpp b/modules/demux/mkv/dispatcher.hpp
index 9b4e3e696b..561c62160f 100644
--- a/modules/demux/mkv/dispatcher.hpp
+++ b/modules/demux/mkv/dispatcher.hpp
@@ -94,9 +94,6 @@ namespace {
typedef PayloadType_ payload_t; \
typedef DispatchType_ dispatch_t; \
typedef struct GroupName_ handler_t; \
- static void* Payload (payload_t& data) { \
- return static_cast<void*> (&data); \
- } \
}; \
struct GroupName_ : GroupName_ ## _base
diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index 0bce8bf1f5..096a5262ed 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -305,7 +305,7 @@ bool matroska_segment_c::ParseSimpleTags( SimpleTag* pout_simple, KaxTagSimple *
vars.out.sub_tags.push_back( st );
}
};
- SimpleTagHandler::Dispatcher().iterate( tag->begin(), tag->end(), SimpleTagHandler::Payload( payload ) );
+ SimpleTagHandler::Dispatcher().iterate( tag->begin(), tag->end(), &payload );
if( pout_simple->tag_name.empty() )
{
@@ -417,7 +417,7 @@ void matroska_segment_c::LoadTags( KaxTags *tags )
}
};
- TargetsHandler::Dispatcher().iterate( targets.begin(), targets.end(), TargetsHandler::Payload( vars ) );
+ TargetsHandler::Dispatcher().iterate( targets.begin(), targets.end(), &vars );
}
E_CASE( KaxTagSimple, entry )
{
@@ -432,7 +432,7 @@ void matroska_segment_c::LoadTags( KaxTags *tags )
}
};
- TagHandler::Dispatcher().iterate( entry.begin(), entry.end(), TagHandler::Payload( payload ) );
+ TagHandler::Dispatcher().iterate( entry.begin(), entry.end(), &payload );
vars.obj->tags.push_back(tag);
}
E_CASE_DEFAULT( el )
@@ -441,7 +441,7 @@ void matroska_segment_c::LoadTags( KaxTags *tags )
}
};
- KaxTagsHandler::Dispatcher().iterate( tags->begin(), tags->end(), KaxTagsHandler::Payload( payload ) );
+ KaxTagsHandler::Dispatcher().iterate( tags->begin(), tags->end(), &payload );
msg_Dbg( &sys.demuxer, "loading tags done." );
}
@@ -506,7 +506,7 @@ bool matroska_segment_c::PreloadClusters(uint64 i_cluster_pos)
if( el == NULL )
break;
- ClusterHandler::Dispatcher().send( el, ClusterHandler::Payload( payload ) );
+ ClusterHandler::Dispatcher().send( el, &payload );
}
}
@@ -1389,7 +1389,7 @@ int matroska_segment_c::BlockGet( KaxBlock * & pp_block, KaxSimpleBlock * & pp_s
case 1:
{
EbmlTypeDispatcher const * dispatcher = dispatchers[i_level - 1];
- dispatcher->send( el, BlockGetHandler_l1::Payload( payload ) );
+ dispatcher->send( el, &payload );
}
break;
diff --git a/modules/demux/mkv/matroska_segment_parse.cpp b/modules/demux/mkv/matroska_segment_parse.cpp
index a2b96970c9..6683eec79c 100644
--- a/modules/demux/mkv/matroska_segment_parse.cpp
+++ b/modules/demux/mkv/matroska_segment_parse.cpp
@@ -409,7 +409,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
}
vars.level += 1;
- dispatcher.iterate( cencs.begin(), cencs.end(), Payload( vars ) );
+ dispatcher.iterate( cencs.begin(), cencs.end(), &vars );
vars.level -= 1;
}
E_CASE( KaxContentEncoding, cenc )
@@ -417,7 +417,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
debug( vars, "Content Encoding" );
vars.level += 1;
- dispatcher.iterate( cenc.begin(), cenc.end(), Payload( vars ) );
+ dispatcher.iterate( cenc.begin(), cenc.end(), &vars );
vars.level -= 1;
}
E_CASE( KaxContentEncodingOrder, encord )
@@ -440,7 +440,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
vars.tk->i_compression_type = MATROSKA_COMPRESSION_ZLIB;
vars.level += 1;
- dispatcher.iterate( compr.begin(), compr.end(), Payload( vars ) );
+ dispatcher.iterate( compr.begin(), compr.end(), &vars );
vars.level -= 1;
}
E_CASE( KaxContentCompAlgo, compalg )
@@ -476,7 +476,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
}
vars.level += 1;
- dispatcher.iterate (tkv.begin (), tkv.end (), Payload( vars ) );
+ dispatcher.iterate (tkv.begin (), tkv.end (), &vars );
vars.level -= 1;
unsigned int i_crop_top = vars.track_video_info.i_crop_top;
@@ -514,7 +514,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
debug( vars, "Track Video Projection" ) ;
vars.level += 1;
- dispatcher.iterate (proj.begin (), proj.end (), Payload( vars ) );
+ dispatcher.iterate (proj.begin (), proj.end (), &vars );
vars.level -= 1;
}
E_CASE( KaxVideoProjectionType, fint )
@@ -640,7 +640,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
msg_Err( vars.p_demuxer, "Video colors elements not allowed for this track" );
} else {
vars.level += 1;
- dispatcher.iterate (colours.begin (), colours.end (), Payload( vars ) );
+ dispatcher.iterate (colours.begin (), colours.end (), &vars );
vars.level -= 1;
}
}
@@ -745,7 +745,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
msg_Err( vars.p_demuxer, "Video metadata elements not allowed for this track" );
} else {
vars.level += 1;
- dispatcher.iterate (mastering.begin (), mastering.end (), Payload( vars ) );
+ dispatcher.iterate (mastering.begin (), mastering.end (), &vars );
vars.level -= 1;
}
}
@@ -806,7 +806,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
msg_Err( vars.p_demuxer, "Audio elements not allowed for this track" );
} else {
vars.level += 1;
- dispatcher.iterate( tka.begin(), tka.end(), Payload( vars ));
+ dispatcher.iterate( tka.begin(), tka.end(), &vars );
vars.level -= 1;
}
}
@@ -842,7 +842,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
}
};
- MetaDataHandlers::Dispatcher().iterate ( m->begin(), m->end(), MetaDataHandlers::Payload( metadata_payload ) );
+ MetaDataHandlers::Dispatcher().iterate ( m->begin(), m->end(), &metadata_payload );
if( p_track->i_number == 0 )
{
@@ -927,8 +927,7 @@ void matroska_segment_c::ParseTracks( KaxTracks *tracks )
}
};
- TrackHandlers::Dispatcher().iterate(
- tracks->begin(), tracks->end(), TrackHandlers::Payload( payload ) );
+ TrackHandlers::Dispatcher().iterate( tracks->begin(), tracks->end(), &payload );
}
/*****************************************************************************
@@ -1083,7 +1082,7 @@ void matroska_segment_c::ParseInfo( KaxInfo *info )
chapter_translation_c *p_translate = new chapter_translation_c();
TranslationHandler::Dispatcher().iterate(
- trans.begin(), trans.end(), TranslationHandler::Payload( p_translate )
+ trans.begin(), trans.end(), &p_translate
);
vars.obj->translations.push_back( p_translate );
@@ -1103,7 +1102,7 @@ void matroska_segment_c::ParseInfo( KaxInfo *info )
}
};
- InfoHandlers::Dispatcher().iterate( m->begin(), m->end(), InfoHandlers::Payload( captures ) );
+ InfoHandlers::Dispatcher().iterate( m->begin(), m->end(), &captures );
if( i_duration != -1 )
i_duration = mtime_t( static_cast<double>( i_duration * i_timescale ) / 10e5 );
@@ -1184,7 +1183,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap
debug( vars, "ChapterDisplay" );
vars.level += 1;
- dispatcher.iterate( chapter_display.begin(), chapter_display.end(), Payload( vars ) );
+ dispatcher.iterate( chapter_display.begin(), chapter_display.end(), &vars );
vars.level -= 1;
}
E_CASE( KaxChapterString, name )
@@ -1257,7 +1256,7 @@ void matroska_segment_c::ParseChapterAtom( int i_level, KaxChapterAtom *ca, chap
}
};
- ChapterAtomHandlers::Dispatcher().iterate( ca->begin(), ca->end(), ChapterAtomHandlers::Payload( payload ) );
+ ChapterAtomHandlers::Dispatcher().iterate( ca->begin(), ca->end(), &payload );
}
/*****************************************************************************
@@ -1392,7 +1391,7 @@ void matroska_segment_c::ParseChapters( KaxChapters *chapters )
msg_Dbg( vars.p_demuxer, "| | | + Unknown (%s)", typeid(el).name() );
}
};
- KaxEditionHandler::Dispatcher().iterate( entry.begin(), entry.end(), KaxEditionHandler::Payload( data ) );
+ KaxEditionHandler::Dispatcher().iterate( entry.begin(), entry.end(), &data );
data.obj->stored_editions.push_back( data.p_edition );
}
@@ -1406,7 +1405,7 @@ void matroska_segment_c::ParseChapters( KaxChapters *chapters )
}
};
- KaxChapterHandler::Dispatcher().iterate( chapters->begin(), chapters->end(), KaxChapterHandler::Payload( *this ) );
+ KaxChapterHandler::Dispatcher().iterate( chapters->begin(), chapters->end(), this );
}
bool matroska_segment_c::ParseCluster( KaxCluster *cluster, bool b_update_start_time, ScopeMode read_fully )
@@ -2058,9 +2057,7 @@ bool matroska_segment_c::TrackInit( mkv_track_t * p_tk )
};
try {
- TrackCodecHandlers::Dispatcher().send( p_tk->codec.c_str(),
- TrackCodecHandlers::Payload( captures )
- );
+ TrackCodecHandlers::Dispatcher().send( p_tk->codec.c_str(), &captures );
}
catch (std::exception const& e)
{
More information about the vlc-commits
mailing list