ffmpeg | branch: master | Andreas Rheinhardt <[email protected]> | 
Thu Mar 25 02:55:53 2021 +0100| [3183eda0fa4861e88c02150b887e2945839a58a1] | 
committer: Andreas Rheinhardt

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 <[email protected]>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3183eda0fa4861e88c02150b887e2945839a58a1
---

 libavcodec/mlpenc.c | 8 ++++----
 1 file 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);

_______________________________________________
ffmpeg-cvslog mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to