Le primidi 11 ventôse, an CCXXIV, Clement Boesch a écrit :
> > > + case H264_PROF_BASELINE:
> > > + switch (vtctx->level) {
> > > + case 0: *profile_level_val =
> > > kVTProfileLevel_H264_Baseline_AutoLevel; break;
^^^^^^^^
> > > + case H264_PROF_MAIN:
> > > + switch (vtctx->level) {
> > > + case 0: *profile_level_val =
> > > kVTProfileLevel_H264_Main_AutoLevel; break;
^^^^
> I don't understand: why don't you put the kVTProfileLevel_H264_* as const
> values? You won't have to do this mapping anymore.
I wondered too, but I found an explanation. Apparently, the wrapped API
combines the profile and level setting into a single constant. We could
replicate that in FFmpeg, but the other codecs use separate profile and
level options.
It could probably be done with an array instead of a switch; I do not know
what results in the most compact code.
Regards,
--
Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
