Using << on an int8_t or uint8_t will output a character rather than a number.
Places that do this need to cast to int to get the desired behavior.
BUG=none
TEST=none
R=henrik.lundin@webrtc.org, pthatcher@webrtc.org, stefan@webrtc.org
Review URL: https://webrtc-codereview.appspot.com/40579004
Cr-Commit-Position: refs/heads/master@{#8223}
git-svn-id: http://webrtc.googlecode.com/svn/trunk@8223 4adac7df-926f-26a2-2b94-8c16560cd09d
diff --git a/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc b/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
index 07c2784..5466c18 100644
--- a/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
+++ b/webrtc/modules/audio_coding/main/acm2/acm_receiver.cc
@@ -274,8 +274,9 @@
int codec_id = RtpHeaderToCodecIndex(*header, incoming_payload);
if (codec_id < 0) {
- LOG_F(LS_ERROR) << "Payload-type " << header->payloadType
- << " is not registered.";
+ LOG_F(LS_ERROR) << "Payload-type "
+ << static_cast<int>(header->payloadType)
+ << " is not registered.";
return -1;
}
assert(codec_id < ACMCodecDB::kMaxNumCodecs);
@@ -340,8 +341,9 @@
if (neteq_->InsertPacket(rtp_header, incoming_payload, length_payload,
receive_timestamp) < 0) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::InsertPacket", header->payloadType) <<
- " Failed to insert packet";
+ LOG_FERR1(LS_ERROR, "AcmReceiver::InsertPacket",
+ static_cast<int>(header->payloadType))
+ << " Failed to insert packet";
return -1;
}
return 0;
@@ -506,7 +508,7 @@
if (neteq_->RemovePayloadType(decoders_[acm_codec_id].payload_type) !=
NetEq::kOK) {
LOG_F(LS_ERROR) << "Cannot remover payload "
- << decoders_[acm_codec_id].payload_type;
+ << static_cast<int>(decoders_[acm_codec_id].payload_type);
return -1;
}
}
@@ -519,8 +521,8 @@
audio_decoder, neteq_decoder, payload_type);
}
if (ret_val != NetEq::kOK) {
- LOG_FERR3(LS_ERROR, "AcmReceiver::AddCodec", acm_codec_id, payload_type,
- channels);
+ LOG_FERR3(LS_ERROR, "AcmReceiver::AddCodec", acm_codec_id,
+ static_cast<int>(payload_type), channels);
// Registration failed, delete the allocated space and set the pointer to
// NULL, for the record.
decoders_[acm_codec_id].registered = false;
@@ -560,7 +562,7 @@
decoders_[n].registered = false;
} else {
LOG_F(LS_ERROR) << "Cannot remove payload "
- << decoders_[n].payload_type;
+ << static_cast<int>(decoders_[n].payload_type);
ret_val = -1;
}
}
@@ -576,7 +578,8 @@
return 0;
}
if (neteq_->RemovePayloadType(payload_type) != NetEq::kOK) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::RemoveCodec", payload_type);
+ LOG_FERR1(LS_ERROR, "AcmReceiver::RemoveCodec",
+ static_cast<int>(payload_type));
return -1;
}
CriticalSectionScoped lock(crit_sect_.get());
@@ -683,7 +686,8 @@
CriticalSectionScoped lock(crit_sect_.get());
int codec_index = PayloadType2CodecIndex(payload_type);
if (codec_index < 0) {
- LOG_FERR1(LS_ERROR, "AcmReceiver::DecoderByPayloadType", payload_type);
+ LOG_FERR1(LS_ERROR, "AcmReceiver::DecoderByPayloadType",
+ static_cast<int>(payload_type));
return -1;
}
memcpy(codec, &ACMCodecDB::database_[codec_index], sizeof(CodecInst));
diff --git a/webrtc/modules/audio_coding/neteq/neteq_impl.cc b/webrtc/modules/audio_coding/neteq/neteq_impl.cc
index 4dd8a6a..537638d 100644
--- a/webrtc/modules/audio_coding/neteq/neteq_impl.cc
+++ b/webrtc/modules/audio_coding/neteq/neteq_impl.cc
@@ -182,7 +182,8 @@
LOG_API2(static_cast<int>(rtp_payload_type), codec);
int ret = decoder_database_->RegisterPayload(rtp_payload_type, codec);
if (ret != DecoderDatabase::kOK) {
- LOG_FERR2(LS_WARNING, RegisterPayload, rtp_payload_type, codec);
+ LOG_FERR2(LS_WARNING, RegisterPayload, static_cast<int>(rtp_payload_type),
+ codec);
switch (ret) {
case DecoderDatabase::kInvalidRtpPayloadType:
error_code_ = kInvalidRtpPayloadType;
@@ -215,7 +216,8 @@
int ret = decoder_database_->InsertExternal(rtp_payload_type, codec,
sample_rate_hz, decoder);
if (ret != DecoderDatabase::kOK) {
- LOG_FERR2(LS_WARNING, InsertExternal, rtp_payload_type, codec);
+ LOG_FERR2(LS_WARNING, InsertExternal, static_cast<int>(rtp_payload_type),
+ codec);
switch (ret) {
case DecoderDatabase::kInvalidRtpPayloadType:
error_code_ = kInvalidRtpPayloadType;
@@ -251,7 +253,7 @@
} else {
error_code_ = kOtherError;
}
- LOG_FERR1(LS_WARNING, Remove, rtp_payload_type);
+ LOG_FERR1(LS_WARNING, Remove, static_cast<int>(rtp_payload_type));
return kFail;
}
@@ -412,7 +414,7 @@
decoder_database_->IsRed(rtp_header.header.payloadType) ||
decoder_database_->IsComfortNoise(rtp_header.header.payloadType)) {
LOG_F(LS_ERROR) << "Sync-packet with an unacceptable payload type "
- << rtp_header.header.payloadType;
+ << static_cast<int>(rtp_header.header.payloadType);
return kSyncPacketNotAccepted;
}
if (first_packet_ ||
@@ -1801,8 +1803,9 @@
packet->payload_length);
}
} else {
- LOG_FERR1(LS_WARNING, GetDecoder, packet->header.payloadType) <<
- "Could not find a decoder for a packet about to be extracted.";
+ LOG_FERR1(LS_WARNING, GetDecoder,
+ static_cast<int>(packet->header.payloadType))
+ << "Could not find a decoder for a packet about to be extracted.";
assert(false);
}
if (packet_duration <= 0) {