Replace RTC_WARN_UNUSED_RESULT with ABSL_MUST_USE_RESULT in audio_coding

Bug: webrtc:12336
Change-Id: Icae229b957c2bfcc410788179a504c576cfde151
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/201736
Reviewed-by: Ivo Creusen <ivoc@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32995}
This commit is contained in:
Danil Chapovalov
2021-01-14 18:35:41 +01:00
committed by Commit Bot
parent 87e9f6e666
commit a86cef7e2c
5 changed files with 15 additions and 11 deletions

View File

@ -368,8 +368,8 @@ rtc_library("ilbc_c") {
"../../rtc_base:rtc_base_approved", "../../rtc_base:rtc_base_approved",
"../../rtc_base:sanitizer", "../../rtc_base:sanitizer",
"../../rtc_base/system:arch", "../../rtc_base/system:arch",
"../../rtc_base/system:unused",
] ]
absl_deps = [ "//third_party/abseil-cpp/absl/base:core_headers" ]
} }
rtc_source_set("isac_common") { rtc_source_set("isac_common") {

View File

@ -23,14 +23,15 @@
#include <stddef.h> #include <stddef.h>
#include <stdint.h> #include <stdint.h>
#include "absl/base/attributes.h"
#include "modules/audio_coding/codecs/ilbc/defines.h" #include "modules/audio_coding/codecs/ilbc/defines.h"
#include "rtc_base/system/unused.h"
/*----------------------------------------------------------------* /*----------------------------------------------------------------*
* Construct decoded vector from codebook and gains. * Construct decoded vector from codebook and gains.
*---------------------------------------------------------------*/ *---------------------------------------------------------------*/
// Returns true on success, false on failure. // Returns true on success, false on failure.
ABSL_MUST_USE_RESULT
bool WebRtcIlbcfix_CbConstruct( bool WebRtcIlbcfix_CbConstruct(
int16_t* decvector, /* (o) Decoded vector */ int16_t* decvector, /* (o) Decoded vector */
const int16_t* index, /* (i) Codebook indices */ const int16_t* index, /* (i) Codebook indices */
@ -38,6 +39,6 @@ bool WebRtcIlbcfix_CbConstruct(
int16_t* mem, /* (i) Buffer for codevector construction */ int16_t* mem, /* (i) Buffer for codevector construction */
size_t lMem, /* (i) Length of buffer */ size_t lMem, /* (i) Length of buffer */
size_t veclen /* (i) Length of vector */ size_t veclen /* (i) Length of vector */
) RTC_WARN_UNUSED_RESULT; );
#endif #endif

View File

@ -21,21 +21,22 @@
#include <stdint.h> #include <stdint.h>
#include "absl/base/attributes.h"
#include "modules/audio_coding/codecs/ilbc/defines.h" #include "modules/audio_coding/codecs/ilbc/defines.h"
#include "rtc_base/system/unused.h"
/*----------------------------------------------------------------* /*----------------------------------------------------------------*
* main decoder function * main decoder function
*---------------------------------------------------------------*/ *---------------------------------------------------------------*/
// Returns 0 on success, -1 on error. // Returns 0 on success, -1 on error.
ABSL_MUST_USE_RESULT
int WebRtcIlbcfix_DecodeImpl( int WebRtcIlbcfix_DecodeImpl(
int16_t* decblock, /* (o) decoded signal block */ int16_t* decblock, /* (o) decoded signal block */
const uint16_t* bytes, /* (i) encoded signal bits */ const uint16_t* bytes, /* (i) encoded signal bits */
IlbcDecoder* iLBCdec_inst, /* (i/o) the decoder state IlbcDecoder* iLBCdec_inst, /* (i/o) the decoder state
structure */ structure */
int16_t mode /* (i) 0: bad packet, PLC, int16_t mode /* (i) 0: bad packet, PLC,
1: normal */ 1: normal */
) RTC_WARN_UNUSED_RESULT; );
#endif #endif

View File

@ -23,8 +23,8 @@
#include <stddef.h> #include <stddef.h>
#include <stdint.h> #include <stdint.h>
#include "absl/base/attributes.h"
#include "modules/audio_coding/codecs/ilbc/defines.h" #include "modules/audio_coding/codecs/ilbc/defines.h"
#include "rtc_base/system/unused.h"
/*----------------------------------------------------------------* /*----------------------------------------------------------------*
* frame residual decoder function (subrutine to iLBC_decode) * frame residual decoder function (subrutine to iLBC_decode)
@ -32,6 +32,7 @@
// Returns true on success, false on failure. In case of failure, the decoder // Returns true on success, false on failure. In case of failure, the decoder
// state may be corrupted and needs resetting. // state may be corrupted and needs resetting.
ABSL_MUST_USE_RESULT
bool WebRtcIlbcfix_DecodeResidual( bool WebRtcIlbcfix_DecodeResidual(
IlbcDecoder* iLBCdec_inst, /* (i/o) the decoder state structure */ IlbcDecoder* iLBCdec_inst, /* (i/o) the decoder state structure */
iLBC_bits* iLBC_encbits, /* (i/o) Encoded bits, which are used iLBC_bits* iLBC_encbits, /* (i/o) Encoded bits, which are used
@ -39,6 +40,6 @@ bool WebRtcIlbcfix_DecodeResidual(
int16_t* decresidual, /* (o) decoded residual frame */ int16_t* decresidual, /* (o) decoded residual frame */
int16_t* syntdenum /* (i) the decoded synthesis filter int16_t* syntdenum /* (i) the decoded synthesis filter
coefficients */ coefficients */
) RTC_WARN_UNUSED_RESULT; );
#endif #endif

View File

@ -23,17 +23,18 @@
#include <stddef.h> #include <stddef.h>
#include <stdint.h> #include <stdint.h>
#include "absl/base/attributes.h"
#include "modules/audio_coding/codecs/ilbc/defines.h" #include "modules/audio_coding/codecs/ilbc/defines.h"
#include "rtc_base/system/unused.h"
// Returns true on success, false on failure. In case of failure, the decoder // Returns true on success, false on failure. In case of failure, the decoder
// state may be corrupted and needs resetting. // state may be corrupted and needs resetting.
ABSL_MUST_USE_RESULT
bool WebRtcIlbcfix_GetCbVec( bool WebRtcIlbcfix_GetCbVec(
int16_t* cbvec, /* (o) Constructed codebook vector */ int16_t* cbvec, /* (o) Constructed codebook vector */
int16_t* mem, /* (i) Codebook buffer */ int16_t* mem, /* (i) Codebook buffer */
size_t index, /* (i) Codebook index */ size_t index, /* (i) Codebook index */
size_t lMem, /* (i) Length of codebook buffer */ size_t lMem, /* (i) Length of codebook buffer */
size_t cbveclen /* (i) Codebook vector length */ size_t cbveclen /* (i) Codebook vector length */
) RTC_WARN_UNUSED_RESULT; );
#endif #endif