diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2021-03-25 02:55:53 +0100 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2021-03-30 12:36:31 +0200 |
| commit | 3183eda0fa4861e88c02150b887e2945839a58a1 (patch) | |
| tree | 6e4c47447515f680336007551f32a2fe697dab8a | |
| parent | 0ab70230f2a597545fb6063e3cebd26bbca93fa6 (diff) | |
| download | ffmpeg-3183eda0fa4861e88c02150b887e2945839a58a1.tar.gz | |
avcodec/mlpenc: Avoid redundant temporary PutBitContext
We are already word-aligned here, so one can just as well flush the main
PutBitContext.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
| -rw-r--r-- | libavcodec/mlpenc.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libavcodec/mlpenc.c b/libavcodec/mlpenc.c index 9782cb8261..40db76f350 100644 --- a/libavcodec/mlpenc.c +++ b/libavcodec/mlpenc.c @@ -1067,7 +1067,7 @@ static uint8_t *write_substrs(MLPEncodeContext *ctx, uint8_t *buf, int buf_size, RestartHeader *rh = &ctx->restart_header [substr]; int substr_restart_frame = restart_frame; uint8_t parity, checksum; - PutBitContext pb, tmpb; + PutBitContext pb; int params_changed; ctx->cur_restart_header = rh; @@ -1117,9 +1117,9 @@ static uint8_t *write_substrs(MLPEncodeContext *ctx, uint8_t *buf, int buf_size, put_bits(&pb, 32, END_OF_STREAM); } - /* Data must be flushed for the checksum and parity to be correct. */ - tmpb = pb; - flush_put_bits(&tmpb); + /* Data must be flushed for the checksum and parity to be correct; + * notice that we already are word-aligned here. */ + flush_put_bits(&pb); parity = ff_mlp_calculate_parity(buf, put_bits_count(&pb) >> 3) ^ 0xa9; checksum = ff_mlp_checksum8 (buf, put_bits_count(&pb) >> 3); |