Skip to content

Commit

Permalink
Refactor prototypes to use the ARG_QEXT macro
Browse files Browse the repository at this point in the history
Change-Id: Iea4e9895785551acc6c73ccb0708a3752858f9fe
Signed-off-by: Jean-Marc Valin <jeanmarcv@google.com>
  • Loading branch information
yunhohuhg authored and jmvalin committed Feb 13, 2025
1 parent e40fd17 commit f2f43c5
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 42 deletions.
17 changes: 5 additions & 12 deletions celt/bands.c
Original file line number Diff line number Diff line change
Expand Up @@ -1143,17 +1143,12 @@ static unsigned quant_partition(struct band_ctx *ctx, celt_norm *X,
/* Finally do the actual quantization */
if (encode)
{
cm = alg_quant(X, N, K, spread, B, ec, gain, ctx->resynth,
#ifdef ENABLE_QEXT
ctx->ext_ec, extra_bits,
#endif
cm = alg_quant(X, N, K, spread, B, ec, gain, ctx->resynth
ARG_QEXT(ctx->ext_ec) ARG_QEXT(extra_bits),
ctx->arch);
} else {
cm = alg_unquant(X, N, K, spread, B, ec, gain
#ifdef ENABLE_QEXT
, ctx->ext_ec, extra_bits
#endif
);
ARG_QEXT(ctx->ext_ec) ARG_QEXT(extra_bits));
}
#ifdef ENABLE_QEXT
} else if (ext_b > 2*N<<BITRES)
Expand Down Expand Up @@ -1611,10 +1606,8 @@ void quant_all_bands(int encode, const CELTMode *m, int start, int end,
int dual_stereo, int intensity, int *tf_res, opus_int32 total_bits,
opus_int32 balance, ec_ctx *ec, int LM, int codedBands,
opus_uint32 *seed, int complexity, int arch, int disable_inv
#ifdef ENABLE_QEXT
, ec_ctx *ext_ec, int *extra_pulses, opus_int32 ext_total_bits, const int *cap
#endif
)
ARG_QEXT(ec_ctx *ext_ec) ARG_QEXT(int *extra_pulses)
ARG_QEXT(opus_int32 ext_total_bits) ARG_QEXT(const int *cap))
{
int i;
opus_int32 remaining_bits;
Expand Down
6 changes: 2 additions & 4 deletions celt/bands.h
Original file line number Diff line number Diff line change
Expand Up @@ -109,10 +109,8 @@ void quant_all_bands(int encode, const CELTMode *m, int start, int end,
int dual_stereo, int intensity, int *tf_res, opus_int32 total_bits,
opus_int32 balance, ec_ctx *ec, int M, int codedBands, opus_uint32 *seed,
int complexity, int arch, int disable_inv
#ifdef ENABLE_QEXT
, ec_ctx *ext_ec, int *extra_pulses, opus_int32 total_ext_bits, const int *cap
#endif
);
ARG_QEXT(ec_ctx *ext_ec) ARG_QEXT(int *extra_pulses)
ARG_QEXT(opus_int32 total_ext_bits) ARG_QEXT(const int *cap));

void anti_collapse(const CELTMode *m, celt_norm *X_,
unsigned char *collapse_masks, int LM, int C, int size, int start,
Expand Down
6 changes: 2 additions & 4 deletions celt/celt_decoder.c
Original file line number Diff line number Diff line change
Expand Up @@ -1393,10 +1393,8 @@ int celt_decode_with_ec_dred(CELTDecoder * OPUS_RESTRICT st, const unsigned char
NULL, pulses, shortBlocks, spread_decision, dual_stereo, intensity, tf_res,
len*(8<<BITRES)-anti_collapse_rsv, balance, dec, LM, codedBands, &st->rng, 0,
st->arch, st->disable_inv
#ifdef ENABLE_QEXT
, &ext_dec, extra_pulses, qext_bytes*(8<<BITRES), cap
#endif
);
ARG_QEXT(&ext_dec) ARG_QEXT(extra_pulses)
ARG_QEXT(qext_bytes*(8<<BITRES)) ARG_QEXT(cap));

#ifdef ENABLE_QEXT
if (qext_mode) {
Expand Down
6 changes: 2 additions & 4 deletions celt/celt_encoder.c
Original file line number Diff line number Diff line change
Expand Up @@ -2585,10 +2585,8 @@ int celt_encode_with_ec(CELTEncoder * OPUS_RESTRICT st, const opus_res * pcm, in
bandE, pulses, shortBlocks, st->spread_decision,
dual_stereo, st->intensity, tf_res, nbCompressedBytes*(8<<BITRES)-anti_collapse_rsv,
balance, enc, LM, codedBands, &st->rng, st->complexity, st->arch, st->disable_inv
#ifdef ENABLE_QEXT
, &ext_enc, extra_pulses, qext_bytes*(8<<BITRES), cap
#endif
);
ARG_QEXT(&ext_enc) ARG_QEXT(extra_pulses)
ARG_QEXT(qext_bytes*(8<<BITRES)) ARG_QEXT(cap));

#ifdef ENABLE_QEXT
if (qext_mode) {
Expand Down
12 changes: 3 additions & 9 deletions celt/vq.c
Original file line number Diff line number Diff line change
Expand Up @@ -526,11 +526,8 @@ static int ec_dec_refine(ec_enc *dec, opus_int32 up, int extra_bits, int use_ent
#endif

unsigned alg_quant(celt_norm *X, int N, int K, int spread, int B, ec_enc *enc,
opus_val32 gain, int resynth,
#ifdef ENABLE_QEXT
ec_enc *ext_enc, int extra_bits,
#endif
int arch)
opus_val32 gain, int resynth
ARG_QEXT(ec_enc *ext_enc) ARG_QEXT(int extra_bits), int arch)
{
VARDECL(int, iy);
opus_val32 yy;
Expand Down Expand Up @@ -600,10 +597,7 @@ unsigned alg_quant(celt_norm *X, int N, int K, int spread, int B, ec_enc *enc,
the final normalised signal in the current band. */
unsigned alg_unquant(celt_norm *X, int N, int K, int spread, int B,
ec_dec *dec, opus_val32 gain
#ifdef ENABLE_QEXT
, ec_enc *ext_dec, int extra_bits
#endif
)
ARG_QEXT(ec_enc *ext_dec) ARG_QEXT(int extra_bits))
{
opus_val32 Ryy;
unsigned collapse_mask;
Expand Down
12 changes: 3 additions & 9 deletions celt/vq.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,11 +74,8 @@ opus_val16 op_pvq_search_c(celt_norm *X, int *iy, int K, int N, int arch);
* @ret A mask indicating which blocks in the band received pulses
*/
unsigned alg_quant(celt_norm *X, int N, int K, int spread, int B, ec_enc *enc,
opus_val32 gain, int resynth,
#ifdef ENABLE_QEXT
ec_enc *ext_enc, int extra_bits,
#endif
int arch);
opus_val32 gain, int resynth
ARG_QEXT(ec_enc *ext_enc) ARG_QEXT(int extra_bits), int arch);

/** Algebraic pulse decoder
* @param X Decoded normalised spectrum (returned)
Expand All @@ -89,10 +86,7 @@ unsigned alg_quant(celt_norm *X, int N, int K, int spread, int B, ec_enc *enc,
*/
unsigned alg_unquant(celt_norm *X, int N, int K, int spread, int B,
ec_dec *dec, opus_val32 gain
#ifdef ENABLE_QEXT
, ec_enc *ext_dec, int extra_bits
#endif
);
ARG_QEXT(ec_enc *ext_dec) ARG_QEXT(int extra_bits));

void renormalise_vector(celt_norm *X, int N, opus_val32 gain, int arch);

Expand Down

0 comments on commit f2f43c5

Please sign in to comment.