Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Unified Diff: webrtc/modules/audio_coding/neteq/include/neteq.h

Issue 2355503002: Stopped using the NetEqDecoder enum internally in NetEq. (Closed)
Patch Set: Clarified comments. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_coding/neteq/include/neteq.h
diff --git a/webrtc/modules/audio_coding/neteq/include/neteq.h b/webrtc/modules/audio_coding/neteq/include/neteq.h
index c07143c062020dd9fd02d46843cf550fd82168e4..8520905b1097b1e6e0c13b7ce57dcfd8ee58275d 100644
--- a/webrtc/modules/audio_coding/neteq/include/neteq.h
+++ b/webrtc/modules/audio_coding/neteq/include/neteq.h
@@ -258,10 +258,10 @@ class NetEq {
// value if we have no decoder for that payload type.
virtual rtc::Optional<CodecInst> GetDecoder(int payload_type) const = 0;
- // Returns the decoder format for the given payload type. Returns null if no
- // such payload type was registered, or if it was registered without
- // providing an SdpAudioFormat.
- virtual const SdpAudioFormat* GetDecoderFormat(int payload_type) const = 0;
+ // Returns the decoder format for the given payload type. Returns empty if no
+ // such payload type was registered.
+ virtual rtc::Optional<SdpAudioFormat> GetDecoderFormat(
+ int payload_type) const = 0;
// Not implemented.
virtual int SetTargetNumberOfChannels() = 0;
« no previous file with comments | « webrtc/modules/audio_coding/neteq/delay_manager.cc ('k') | webrtc/modules/audio_coding/neteq/mock/mock_audio_decoder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698