mirror of
https://github.com/nyanmisaka/ffmpeg-rockchip.git
synced 2025-10-07 01:36:16 +08:00
cbs: Remove useless initializations
Up until now, a temporary variable was used and initialized every time a value was read in CBS; if reading turned out to be successfull, this value was overwritten (without having ever been looked at) with the value read if reading was successfull; on failure the variable wasn't touched either. Therefore these initializations can be and have been removed. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
This commit is contained in:

committed by
Mark Thompson

parent
c2a91645c5
commit
b71a0367a6
@@ -574,7 +574,7 @@ static size_t cbs_av1_get_payload_bytes_left(GetBitContext *gbc)
|
||||
#define RWContext GetBitContext
|
||||
|
||||
#define xf(width, name, var, range_min, range_max, subs, ...) do { \
|
||||
uint32_t value = range_min; \
|
||||
uint32_t value; \
|
||||
CHECK(ff_cbs_read_unsigned(ctx, rw, width, #name, \
|
||||
SUBSCRIPTS(subs, __VA_ARGS__), \
|
||||
&value, range_min, range_max)); \
|
||||
@@ -582,7 +582,7 @@ static size_t cbs_av1_get_payload_bytes_left(GetBitContext *gbc)
|
||||
} while (0)
|
||||
|
||||
#define xsu(width, name, var, subs, ...) do { \
|
||||
int32_t value = 0; \
|
||||
int32_t value; \
|
||||
CHECK(ff_cbs_read_signed(ctx, rw, width, #name, \
|
||||
SUBSCRIPTS(subs, __VA_ARGS__), &value, \
|
||||
MIN_INT_BITS(width), \
|
||||
@@ -591,27 +591,27 @@ static size_t cbs_av1_get_payload_bytes_left(GetBitContext *gbc)
|
||||
} while (0)
|
||||
|
||||
#define uvlc(name, range_min, range_max) do { \
|
||||
uint32_t value = range_min; \
|
||||
uint32_t value; \
|
||||
CHECK(cbs_av1_read_uvlc(ctx, rw, #name, \
|
||||
&value, range_min, range_max)); \
|
||||
current->name = value; \
|
||||
} while (0)
|
||||
|
||||
#define ns(max_value, name, subs, ...) do { \
|
||||
uint32_t value = 0; \
|
||||
uint32_t value; \
|
||||
CHECK(cbs_av1_read_ns(ctx, rw, max_value, #name, \
|
||||
SUBSCRIPTS(subs, __VA_ARGS__), &value)); \
|
||||
current->name = value; \
|
||||
} while (0)
|
||||
|
||||
#define increment(name, min, max) do { \
|
||||
uint32_t value = 0; \
|
||||
uint32_t value; \
|
||||
CHECK(cbs_av1_read_increment(ctx, rw, min, max, #name, &value)); \
|
||||
current->name = value; \
|
||||
} while (0)
|
||||
|
||||
#define subexp(name, max, subs, ...) do { \
|
||||
uint32_t value = 0; \
|
||||
uint32_t value; \
|
||||
CHECK(cbs_av1_read_subexp(ctx, rw, max, #name, \
|
||||
SUBSCRIPTS(subs, __VA_ARGS__), &value)); \
|
||||
current->name = value; \
|
||||
@@ -629,7 +629,7 @@ static size_t cbs_av1_get_payload_bytes_left(GetBitContext *gbc)
|
||||
} while (0)
|
||||
|
||||
#define leb128(name) do { \
|
||||
uint64_t value = 0; \
|
||||
uint64_t value; \
|
||||
CHECK(cbs_av1_read_leb128(ctx, rw, #name, &value)); \
|
||||
current->name = value; \
|
||||
} while (0)
|
||||
|
Reference in New Issue
Block a user