aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--api/audio_codecs/audio_format.cc33
-rw-r--r--api/audio_codecs/audio_format.h6
-rw-r--r--api/rtcerror.cc4
-rw-r--r--api/rtcerror.h12
-rw-r--r--api/rtptransceiverinterface.h3
-rw-r--r--common_types.h10
-rw-r--r--media/base/codec.h22
-rw-r--r--modules/video_coding/codecs/h264/include/h264_globals.h5
-rw-r--r--pc/rtpmediautils.h8
-rw-r--r--pc/rtptransceiver.cc4
-rw-r--r--rtc_base/ipaddress.cc9
-rw-r--r--rtc_base/ipaddress.h10
-rw-r--r--rtc_base/socketaddress.cc5
-rw-r--r--rtc_base/socketaddress.h10
14 files changed, 34 insertions, 107 deletions
diff --git a/api/audio_codecs/audio_format.cc b/api/audio_codecs/audio_format.cc
index 82c166f5c0..9db5ce01dc 100644
--- a/api/audio_codecs/audio_format.cc
+++ b/api/audio_codecs/audio_format.cc
@@ -68,20 +68,6 @@ void swap(SdpAudioFormat& a, SdpAudioFormat& b) {
swap(a.parameters, b.parameters);
}
-std::ostream& operator<<(std::ostream& os, const SdpAudioFormat& saf) {
- os << "{name: " << saf.name;
- os << ", clockrate_hz: " << saf.clockrate_hz;
- os << ", num_channels: " << saf.num_channels;
- os << ", parameters: {";
- const char* sep = "";
- for (const auto& kv : saf.parameters) {
- os << sep << kv.first << ": " << kv.second;
- sep = ", ";
- }
- os << "}}";
- return os;
-}
-
AudioCodecInfo::AudioCodecInfo(int sample_rate_hz,
size_t num_channels,
int bitrate_bps)
@@ -108,23 +94,4 @@ AudioCodecInfo::AudioCodecInfo(int sample_rate_hz,
RTC_DCHECK_GE(max_bitrate_bps, default_bitrate_bps);
}
-std::ostream& operator<<(std::ostream& os, const AudioCodecInfo& aci) {
- os << "{sample_rate_hz: " << aci.sample_rate_hz;
- os << ", num_channels: " << aci.num_channels;
- os << ", default_bitrate_bps: " << aci.default_bitrate_bps;
- os << ", min_bitrate_bps: " << aci.min_bitrate_bps;
- os << ", max_bitrate_bps: " << aci.max_bitrate_bps;
- os << ", allow_comfort_noise: " << aci.allow_comfort_noise;
- os << ", supports_network_adaption: " << aci.supports_network_adaption;
- os << "}";
- return os;
-}
-
-std::ostream& operator<<(std::ostream& os, const AudioCodecSpec& acs) {
- os << "{format: " << acs.format;
- os << ", info: " << acs.info;
- os << "}";
- return os;
-}
-
} // namespace webrtc
diff --git a/api/audio_codecs/audio_format.h b/api/audio_codecs/audio_format.h
index 2a85c6f553..553ab8fb25 100644
--- a/api/audio_codecs/audio_format.h
+++ b/api/audio_codecs/audio_format.h
@@ -12,7 +12,6 @@
#define API_AUDIO_CODECS_AUDIO_FORMAT_H_
#include <map>
-#include <ostream>
#include <string>
#include <utility>
@@ -62,7 +61,6 @@ struct SdpAudioFormat {
};
void swap(SdpAudioFormat& a, SdpAudioFormat& b);
-std::ostream& operator<<(std::ostream& os, const SdpAudioFormat& saf);
// Information about how an audio format is treated by the codec implementation.
// Contains basic information, such as sample rate and number of channels, which
@@ -121,8 +119,6 @@ struct AudioCodecInfo {
// network conditions.
};
-std::ostream& operator<<(std::ostream& os, const AudioCodecInfo& aci);
-
// AudioCodecSpec ties an audio format to specific information about the codec
// and its implementation.
struct AudioCodecSpec {
@@ -136,8 +132,6 @@ struct AudioCodecSpec {
AudioCodecInfo info;
};
-std::ostream& operator<<(std::ostream& os, const AudioCodecSpec& acs);
-
} // namespace webrtc
#endif // API_AUDIO_CODECS_AUDIO_FORMAT_H_
diff --git a/api/rtcerror.cc b/api/rtcerror.cc
index 5a8ba033e8..55ac15e196 100644
--- a/api/rtcerror.cc
+++ b/api/rtcerror.cc
@@ -93,10 +93,6 @@ void RTCError::set_message(std::string&& message) {
}
}
-std::ostream& operator<<(std::ostream& stream, RTCErrorType error) {
- return stream << ToString(error);
-}
-
// TODO(jonasolsson): Change to use absl::string_view when it's available.
std::string ToString(RTCErrorType error) {
int index = static_cast<int>(error);
diff --git a/api/rtcerror.h b/api/rtcerror.h
index d7dec29d43..c87ce91650 100644
--- a/api/rtcerror.h
+++ b/api/rtcerror.h
@@ -11,7 +11,9 @@
#ifndef API_RTCERROR_H_
#define API_RTCERROR_H_
+#ifdef UNIT_TEST
#include <ostream>
+#endif // UNIT_TEST
#include <string>
#include <utility> // For std::move.
@@ -143,10 +145,16 @@ class RTCError {
// error type.
//
// Only intended to be used for logging/disagnostics.
-std::ostream& operator<<(std::ostream& stream, RTCErrorType error);
-
std::string ToString(RTCErrorType error);
+#ifdef UNIT_TEST
+inline std::ostream& operator<<( // no-presubmit-check TODO(webrtc:8982)
+ std::ostream& stream, // no-presubmit-check TODO(webrtc:8982)
+ RTCErrorType error) {
+ return stream << ToString(error);
+}
+#endif // UNIT_TEST
+
// Helper macro that can be used by implementations to create an error with a
// message and log it. |message| should be a string literal or movable
// std::string.
diff --git a/api/rtptransceiverinterface.h b/api/rtptransceiverinterface.h
index 78055797d2..7d2a1dffe2 100644
--- a/api/rtptransceiverinterface.h
+++ b/api/rtptransceiverinterface.h
@@ -30,9 +30,6 @@ enum class RtpTransceiverDirection {
kInactive
};
-// This is provided as a debugging aid. The format of the output is unspecified.
-std::ostream& operator<<(std::ostream& os, RtpTransceiverDirection direction);
-
// Structure for initializing an RtpTransceiver in a call to
// PeerConnectionInterface::AddTransceiver.
// https://w3c.github.io/webrtc-pc/#dom-rtcrtptransceiverinit
diff --git a/common_types.h b/common_types.h
index ae47811f73..580f83ac4c 100644
--- a/common_types.h
+++ b/common_types.h
@@ -223,16 +223,6 @@ struct CodecInst {
}
bool operator!=(const CodecInst& other) const { return !(*this == other); }
-
- friend std::ostream& operator<<(std::ostream& os, const CodecInst& ci) {
- os << "{pltype: " << ci.pltype;
- os << ", plname: " << ci.plname;
- os << ", plfreq: " << ci.plfreq;
- os << ", pacsize: " << ci.pacsize;
- os << ", channels: " << ci.channels;
- os << ", rate: " << ci.rate << "}";
- return os;
- }
};
// RTP
diff --git a/media/base/codec.h b/media/base/codec.h
index 30844c7120..7bbb7db850 100644
--- a/media/base/codec.h
+++ b/media/base/codec.h
@@ -149,28 +149,6 @@ struct AudioCodec : public Codec {
}
};
-inline std::ostream& operator<<(std::ostream& os, const AudioCodec& ac) {
- os << "{id: " << ac.id;
- os << ", name: " << ac.name;
- os << ", clockrate: " << ac.clockrate;
- os << ", bitrate: " << ac.bitrate;
- os << ", channels: " << ac.channels;
- os << ", params: {";
- const char* sep = "";
- for (const auto& kv : ac.params) {
- os << sep << kv.first << ": " << kv.second;
- sep = ", ";
- }
- os << "}, feedback_params: {";
- sep = "";
- for (const FeedbackParam& fp : ac.feedback_params.params()) {
- os << sep << fp.id() << ": " << fp.param();
- sep = ", ";
- }
- os << "}}";
- return os;
-}
-
struct VideoCodec : public Codec {
// Creates a codec with the given parameters.
VideoCodec(int id, const std::string& name);
diff --git a/modules/video_coding/codecs/h264/include/h264_globals.h b/modules/video_coding/codecs/h264/include/h264_globals.h
index 365dbca5d2..bfeba673a9 100644
--- a/modules/video_coding/codecs/h264/include/h264_globals.h
+++ b/modules/video_coding/codecs/h264/include/h264_globals.h
@@ -53,11 +53,6 @@ inline std::string ToString(H264PacketizationMode mode) {
return "";
}
-inline std::ostream& operator<<(std::ostream& stream,
- H264PacketizationMode mode) {
- return stream << ToString(mode);
-}
-
struct NaluInfo {
uint8_t type;
int sps_id;
diff --git a/pc/rtpmediautils.h b/pc/rtpmediautils.h
index 6de6f8f0af..5f02d89907 100644
--- a/pc/rtpmediautils.h
+++ b/pc/rtpmediautils.h
@@ -44,6 +44,14 @@ RtpTransceiverDirection RtpTransceiverDirectionWithRecvSet(
// Returns an unspecified string representation of the given direction.
const char* RtpTransceiverDirectionToString(RtpTransceiverDirection direction);
+#ifdef UNIT_TEST
+inline std::ostream& operator<<( // no-presubmit-check TODO(webrtc:8982)
+ std::ostream& os, // no-presubmit-check TODO(webrtc:8982)
+ RtpTransceiverDirection direction) {
+ return os << RtpTransceiverDirectionToString(direction);
+}
+#endif // UNIT_TEST
+
} // namespace webrtc
#endif // PC_RTPMEDIAUTILS_H_
diff --git a/pc/rtptransceiver.cc b/pc/rtptransceiver.cc
index 41ee487642..37770b524d 100644
--- a/pc/rtptransceiver.cc
+++ b/pc/rtptransceiver.cc
@@ -16,10 +16,6 @@
namespace webrtc {
-std::ostream& operator<<(std::ostream& os, RtpTransceiverDirection direction) {
- return os << RtpTransceiverDirectionToString(direction);
-}
-
RtpTransceiver::RtpTransceiver(cricket::MediaType media_type)
: unified_plan_(false), media_type_(media_type) {
RTC_DCHECK(media_type == cricket::MEDIA_TYPE_AUDIO ||
diff --git a/rtc_base/ipaddress.cc b/rtc_base/ipaddress.cc
index 3f7698acb3..e6a02c4a8a 100644
--- a/rtc_base/ipaddress.cc
+++ b/rtc_base/ipaddress.cc
@@ -120,11 +120,6 @@ bool IPAddress::operator <(const IPAddress &other) const {
return false;
}
-std::ostream& operator<<(std::ostream& os, const IPAddress& ip) {
- os << ip.ToString();
- return os;
-}
-
in6_addr IPAddress::ipv6_address() const {
return u_.ip6;
}
@@ -216,10 +211,6 @@ const InterfaceAddress& InterfaceAddress::operator=(
return *this;
}
-std::ostream& operator<<(std::ostream& os, const InterfaceAddress& ip) {
- return os << ip.ToString();
-}
-
std::string InterfaceAddress::ToString() const {
std::string result = IPAddress::ToString();
diff --git a/rtc_base/ipaddress.h b/rtc_base/ipaddress.h
index aa8bb1a320..c965cf14e7 100644
--- a/rtc_base/ipaddress.h
+++ b/rtc_base/ipaddress.h
@@ -83,7 +83,13 @@ class IPAddress {
bool operator!=(const IPAddress& other) const;
bool operator <(const IPAddress& other) const;
bool operator >(const IPAddress& other) const;
- friend std::ostream& operator<<(std::ostream& os, const IPAddress& addr);
+
+#ifdef UNIT_TEST
+ inline std::ostream& operator<<( // no-presubmit-check TODO(webrtc:8982)
+ std::ostream& os) { // no-presubmit-check TODO(webrtc:8982)
+ return os << ToString();
+ }
+#endif // UNIT_TEST
int family() const { return family_; }
in_addr ipv4_address() const;
@@ -141,8 +147,6 @@ class InterfaceAddress : public IPAddress {
bool operator!=(const InterfaceAddress& other) const;
int ipv6_flags() const { return ipv6_flags_; }
- friend std::ostream& operator<<(std::ostream& os,
- const InterfaceAddress& addr);
std::string ToString() const;
diff --git a/rtc_base/socketaddress.cc b/rtc_base/socketaddress.cc
index bddb7bcfd0..a58c001d32 100644
--- a/rtc_base/socketaddress.cc
+++ b/rtc_base/socketaddress.cc
@@ -199,11 +199,6 @@ bool SocketAddress::FromString(const std::string& str) {
return true;
}
-std::ostream& operator<<(std::ostream& os, const SocketAddress& addr) {
- os << addr.HostAsURIString() << ":" << addr.port();
- return os;
-}
-
bool SocketAddress::IsAnyIP() const {
return IPIsAny(ip_);
}
diff --git a/rtc_base/socketaddress.h b/rtc_base/socketaddress.h
index d58eed8428..90919c2318 100644
--- a/rtc_base/socketaddress.h
+++ b/rtc_base/socketaddress.h
@@ -13,6 +13,9 @@
#include <iosfwd>
#include <string>
+#ifdef UNIT_TEST
+#include <ostream> // no-presubmit-check TODO(webrtc:8982)
+#endif // UNIT_TEST
#include <vector>
#include "rtc_base/basictypes.h"
#include "rtc_base/ipaddress.h"
@@ -126,7 +129,12 @@ class SocketAddress {
// Parses hostname:port and [hostname]:port.
bool FromString(const std::string& str);
- friend std::ostream& operator<<(std::ostream& os, const SocketAddress& addr);
+#ifdef UNIT_TEST
+ inline std::ostream& operator<<( // no-presubmit-check TODO(webrtc:8982)
+ std::ostream& os) { // no-presubmit-check TODO(webrtc:8982)
+ return os << HostAsURIString() << ":" << port();
+ }
+#endif // UNIT_TEST
// Determines whether this represents a missing / any IP address.
// That is, 0.0.0.0 or ::.
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-07 22:08:08 +0000