| Message ID | GV1P250MB0737AB31294010EE180AF5828F032@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM |
|---|---|
| State | Accepted |
| Commit | 908a73df31ff4bd6c82535022152c634b4312ba5 |
| Headers | show |
| Series | [FFmpeg-devel,01/15] avcodec/mpegvideo_enc: Don't update current_picture unnecessarily | expand |
| Context | Check | Description |
|---|---|---|
| andriy/make_x86 | success | Make finished |
| andriy/make_fate_x86 | success | Make fate finished |
Andreas Rheinhardt: > current_picture is not changed after frame_start() at all > and it therefore does not need to be updated (i.e. copied to the > slice thread contexts) a second time. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> > --- > libavcodec/mpegvideo_enc.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c > index d1b1917824..0e3255c0fb 100644 > --- a/libavcodec/mpegvideo_enc.c > +++ b/libavcodec/mpegvideo_enc.c > @@ -251,7 +251,6 @@ static void update_duplicate_context_after_me(MpegEncContext *dst, > { > #define COPY(a) dst->a= src->a > COPY(pict_type); > - COPY(current_picture); > COPY(f_code); > COPY(b_code); > COPY(qscale); Will apply this patchset tonight unless there are objections. - Andreas
diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index d1b1917824..0e3255c0fb 100644 --- a/libavcodec/mpegvideo_enc.c +++ b/libavcodec/mpegvideo_enc.c @@ -251,7 +251,6 @@ static void update_duplicate_context_after_me(MpegEncContext *dst, { #define COPY(a) dst->a= src->a COPY(pict_type); - COPY(current_picture); COPY(f_code); COPY(b_code); COPY(qscale);
current_picture is not changed after frame_start() at all and it therefore does not need to be updated (i.e. copied to the slice thread contexts) a second time. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> --- libavcodec/mpegvideo_enc.c | 1 - 1 file changed, 1 deletion(-)