On Sat, May 4, 2019 at 2:15 PM Luca Barbato <lu_z...@gentoo.org> wrote:
>
> From: James Almer <jamr...@gmail.com>
>
> before:
> 419022 decicycles in assemble_fragment, 2047 runs, 1 skips
>
> after:
> 104621 decicycles in assemble_fragment, 2045 runs, 3 skips
>
> Benched with a 2 minutes long 720x480 DVD mpeg2 sample.
>
> Reviewed-by: Michael Niedermayer <mich...@niedermayer.cc>
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> libavcodec/cbs_mpeg2.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/cbs_mpeg2.c b/libavcodec/cbs_mpeg2.c
> index 7fa13c64fa..7f484267fa 100644
> --- a/libavcodec/cbs_mpeg2.c
> +++ b/libavcodec/cbs_mpeg2.c
> @@ -362,7 +362,7 @@ static int
> cbs_mpeg2_assemble_fragment(CodedBitstreamContext *ctx,
> CodedBitstreamFragment *frag)
> {
> uint8_t *data;
> - size_t size, dp, sp;
> + size_t size, dp;
> int i;
>
> size = 0;
> @@ -382,8 +382,8 @@ static int
> cbs_mpeg2_assemble_fragment(CodedBitstreamContext *ctx,
> data[dp++] = 0;
> data[dp++] = 1;
>
> - for (sp = 0; sp < unit->data_size; sp++)
> - data[dp++] = unit->data[sp];
> + memcpy(data + dp, unit->data, unit->data_size);
> + dp += unit->data_size;
> }
>
> av_assert0(dp == size);
> --
> 2.12.2
>
OK
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
