mbox series

[FFmpeg-devel,v8,0/4] Fix mpeg1/2 stream copy

Message ID 20200305081724.308-1-nicolas.gaullier@cji.paris
Headers show
Series Fix mpeg1/2 stream copy
Related show

Message

Gaullier Nicolas March 5, 2020, 8:17 a.m. UTC
I have not received any feedback yet.
I repost here with all 4 patches, maybe it is easier for the review
- 1/4: fix mpeg12 decoder use of avctx->rc_buffer_size with is forbidden by API doc
- 2/4,3/4,4/4: fix mpeg1/2 stream copy
Thank you

Nicolas Gaullier (4):
  avcodec/mpeg12dec: Do not alter avctx->rc_buffer_size
  avformat/utils: Make find_stream_info get side data from codec context
  avcodec/utils: Fix ff_add_cpb_side_data() add twice
  avcodec/mpeg12dec: Add CPB coded side data

 libavcodec/mpeg12dec.c       | 16 ++++++++++++----
 libavcodec/utils.c           |  5 +++++
 libavformat/utils.c          | 18 ++++++++++++++++++
 tests/ref/fate/mxf-probe-d10 |  3 +++
 tests/ref/fate/ts-demux      |  2 +-
 5 files changed, 39 insertions(+), 5 deletions(-)

Comments

Gaullier Nicolas March 10, 2020, 10:27 a.m. UTC | #1
> De : Nicolas Gaullier <nicolas.gaullier@cji.paris>
> Envoyé : jeudi 5 mars 2020 09:17
> À : ffmpeg-devel@ffmpeg.org
> Cc : Gaullier Nicolas <nicolas.gaullier@cji.paris>
> Objet : [PATCH v8 0/4] Fix mpeg1/2 stream copy
> 
> I have not received any feedback yet.
> I repost here with all 4 patches, maybe it is easier for the review
> - 1/4: fix mpeg12 decoder use of avctx->rc_buffer_size with is forbidden by API doc
> - 2/4,3/4,4/4: fix mpeg1/2 stream copy
> Thank you
> 
> Nicolas Gaullier (4):
>   avcodec/mpeg12dec: Do not alter avctx->rc_buffer_size
>   avformat/utils: Make find_stream_info get side data from codec context
>   avcodec/utils: Fix ff_add_cpb_side_data() add twice
>   avcodec/mpeg12dec: Add CPB coded side data
> 
>  libavcodec/mpeg12dec.c       | 16 ++++++++++++----
>  libavcodec/utils.c           |  5 +++++
>  libavformat/utils.c          | 18 ++++++++++++++++++
>  tests/ref/fate/mxf-probe-d10 |  3 +++
>  tests/ref/fate/ts-demux      |  2 +-
>  5 files changed, 39 insertions(+), 5 deletions(-)
> 
> --
> 2.25.0.windows.1

ping ?
https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=464

Nicolas
Anton Khirnov March 18, 2020, 8:24 a.m. UTC | #2
Hi,
sorry for the delay.
Set seems reasonable and nobody seems to object, so will push soonish.