[FFmpeg-devel] [PATCH] opus/matroska: Adding support for End Trimming in demuxer/decoder
nicolas.george at normalesup.org
Thu Sep 5 21:33:44 CEST 2013
L'octidi 18 fructidor, an CCXXI, Michael Niedermayer a écrit :
> i confirm this fixes the issues
> anyone else has any comments about the patchset ?
> nicolas ? you worked with (lib)opus IIRC
I worked on the decoder, not the encoder. Assuming I understand correctly
what ff_af_queue does, the libopusenc part looks fine (except style:
"uint8_t* side_data" -> "uint8_t *side_data" and "side_data == NULL" ->
I can not comment usefully on the Matroska part, though. It looks fine at
first glance, but I would not spot a subtle mistake.
The "(side_data_size && additional_id == 1)" and "(side_data_size &&
additional_id == 1) || discard_padding" tests look a bit fragile, and may
become a maintenance burden. Maybe checking block_group to see if it was
opened or setting a flag at the same time would be cleaner. But that does
not change the logic of the code.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel