[FFmpeg-cvslog] avcodec/rv34: Don't use MpegEncContext.gb
Andreas Rheinhardt
git at videolan.org
Thu Jul 3 21:55:34 EEST 2025
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Sun Jun 15 23:52:30 2025 +0200| [0e6d59871f17eb200b4d85bca5fb8c8b57441959] | committer: Andreas Rheinhardt
avcodec/rv34: Don't use MpegEncContext.gb
This is in preparation for removing MpegEncContext.gb.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=0e6d59871f17eb200b4d85bca5fb8c8b57441959
---
libavcodec/rv30.c | 2 +-
libavcodec/rv34.c | 28 ++++++++++++++--------------
libavcodec/rv34.h | 2 ++
libavcodec/rv40.c | 2 +-
4 files changed, 18 insertions(+), 16 deletions(-)
diff --git a/libavcodec/rv30.c b/libavcodec/rv30.c
index 351276995b..e0b5395fa6 100644
--- a/libavcodec/rv30.c
+++ b/libavcodec/rv30.c
@@ -116,7 +116,7 @@ static int rv30_decode_mb_info(RV34DecContext *r)
static const int rv30_p_types[6] = { RV34_MB_SKIP, RV34_MB_P_16x16, RV34_MB_P_8x8, -1, RV34_MB_TYPE_INTRA, RV34_MB_TYPE_INTRA16x16 };
static const int rv30_b_types[6] = { RV34_MB_SKIP, RV34_MB_B_DIRECT, RV34_MB_B_FORWARD, RV34_MB_B_BACKWARD, RV34_MB_TYPE_INTRA, RV34_MB_TYPE_INTRA16x16 };
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
unsigned code = get_interleaved_ue_golomb(gb);
if (code > 11) {
diff --git a/libavcodec/rv34.c b/libavcodec/rv34.c
index 5dcbe9a222..73fd16b3e2 100644
--- a/libavcodec/rv34.c
+++ b/libavcodec/rv34.c
@@ -363,7 +363,7 @@ static inline RV34VLC* choose_vlc_set(int quant, int mod, int type)
static int rv34_decode_intra_mb_header(RV34DecContext *r, int8_t *intra_types)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int mb_pos = s->mb_x + s->mb_y * s->mb_stride;
int t;
@@ -398,7 +398,7 @@ static int rv34_decode_intra_mb_header(RV34DecContext *r, int8_t *intra_types)
static int rv34_decode_inter_mb_header(RV34DecContext *r, int8_t *intra_types)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int mb_pos = s->mb_x + s->mb_y * s->mb_stride;
int i, t;
@@ -871,7 +871,7 @@ static const int num_mvs[RV34_MB_TYPES] = { 0, 0, 1, 4, 1, 1, 0, 0, 2, 2, 2, 1 }
static int rv34_decode_mv(RV34DecContext *r, int block_type)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int i, j, k, l;
int mv_pos = s->mb_x * 2 + s->mb_y * 2 * s->b8_stride;
int next_bt;
@@ -1031,7 +1031,7 @@ static inline void rv34_process_block(RV34DecContext *r,
{
MpegEncContext *s = &r->s;
int16_t *ptr = s->block[0];
- int has_ac = rv34_decode_block(ptr, &s->gb, r->cur_vlcs,
+ int has_ac = rv34_decode_block(ptr, &r->gb, r->cur_vlcs,
fc, sc, q_dc, q_ac, q_ac);
if(has_ac){
r->rdsp.rv34_idct_add(pdst, stride, ptr);
@@ -1045,7 +1045,7 @@ static void rv34_output_i16x16(RV34DecContext *r, int8_t *intra_types, int cbp)
{
LOCAL_ALIGNED_16(int16_t, block16, [16]);
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int q_dc = rv34_qscale_tab[ r->luma_dc_quant_i[s->qscale] ],
q_ac = rv34_qscale_tab[s->qscale];
uint8_t *dst = s->dest[0];
@@ -1213,7 +1213,7 @@ static int rv34_set_deblock_coef(RV34DecContext *r)
static int rv34_decode_inter_macroblock(RV34DecContext *r, int8_t *intra_types)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
uint8_t *dst = s->dest[0];
int16_t *ptr = s->block[0];
int mb_pos = s->mb_x + s->mb_y * s->mb_stride;
@@ -1367,8 +1367,8 @@ static int check_slice_end(RV34DecContext *r, MpegEncContext *s)
return 1;
if(r->s.mb_skip_run > 1)
return 0;
- bits = get_bits_left(&s->gb);
- if(bits <= 0 || (bits < 8 && !show_bits(&s->gb, bits)))
+ bits = get_bits_left(&r->gb);
+ if (bits <= 0 || (bits < 8 && !show_bits(&r->gb, bits)))
return 1;
return 0;
}
@@ -1424,11 +1424,11 @@ static int rv34_decoder_realloc(RV34DecContext *r)
static int rv34_decode_slice(RV34DecContext *r, int end, const uint8_t* buf, int buf_size)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int mb_pos, slice_type;
int res;
- init_get_bits(&r->s.gb, buf, buf_size*8);
+ init_get_bits(gb, buf, buf_size*8);
res = r->parse_slice_header(r, gb, &r->si);
if(res < 0){
av_log(s->avctx, AV_LOG_ERROR, "Incorrect or unknown slice header\n");
@@ -1651,8 +1651,8 @@ int ff_rv34_decode_frame(AVCodecContext *avctx, AVFrame *pict,
av_log(avctx, AV_LOG_ERROR, "Slice offset is invalid\n");
return AVERROR_INVALIDDATA;
}
- init_get_bits(&s->gb, buf+offset, (buf_size-offset)*8);
- if(r->parse_slice_header(r, &r->s.gb, &si) < 0 || si.start){
+ init_get_bits(&r->gb, buf+offset, (buf_size-offset)*8);
+ if (r->parse_slice_header(r, &r->gb, &si) < 0 || si.start) {
av_log(avctx, AV_LOG_ERROR, "First slice header is incorrect\n");
return AVERROR_INVALIDDATA;
}
@@ -1781,8 +1781,8 @@ int ff_rv34_decode_frame(AVCodecContext *avctx, AVFrame *pict,
av_log(avctx, AV_LOG_ERROR, "Slice offset is invalid\n");
break;
}
- init_get_bits(&s->gb, buf+offset1, (buf_size-offset1)*8);
- if(r->parse_slice_header(r, &r->s.gb, &si) < 0){
+ init_get_bits(&r->gb, buf+offset1, (buf_size-offset1)*8);
+ if (r->parse_slice_header(r, &r->gb, &si) < 0) {
size = offset2 - offset;
}else
r->si.end = si.start;
diff --git a/libavcodec/rv34.h b/libavcodec/rv34.h
index 6fe1f8087d..485a034f9e 100644
--- a/libavcodec/rv34.h
+++ b/libavcodec/rv34.h
@@ -30,6 +30,7 @@
#include "libavutil/mem_internal.h"
#include "avcodec.h"
+#include "get_bits.h"
#include "mpegvideo.h"
#include "h264pred.h"
@@ -85,6 +86,7 @@ typedef struct SliceInfo{
/** decoder context */
typedef struct RV34DecContext{
MpegEncContext s;
+ GetBitContext gb;
RV34DSPContext rdsp;
int8_t *intra_types_hist;///< old block types, used for prediction
int8_t *intra_types; ///< block types
diff --git a/libavcodec/rv40.c b/libavcodec/rv40.c
index d28e02c2d1..79e99e9efd 100644
--- a/libavcodec/rv40.c
+++ b/libavcodec/rv40.c
@@ -226,7 +226,7 @@ static int rv40_decode_intra_types(RV34DecContext *r, GetBitContext *gb, int8_t
static int rv40_decode_mb_info(RV34DecContext *r)
{
MpegEncContext *s = &r->s;
- GetBitContext *gb = &s->gb;
+ GetBitContext *const gb = &r->gb;
int q, i;
int prev_type = 0;
int mb_pos = s->mb_x + s->mb_y * s->mb_stride;
More information about the ffmpeg-cvslog
mailing list