Remove voice_engine_ member variable and GetVoiceEngine() from ViEChannelManager.
This is dead code right now and since the implementation of GetVoiceEngine() grabbed a lock and returned a raw pointer, it's not to be trusted anyway :) R=mflodman@webrtc.org BUG= Review URL: https://webrtc-codereview.appspot.com/37869005 Cr-Commit-Position: refs/heads/master@{#8306} git-svn-id: http://webrtc.googlecode.com/svn/trunk@8306 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
parent
5b8f3e0206
commit
8e612aba60
@ -38,7 +38,6 @@ ViEChannelManager::ViEChannelManager(
|
|||||||
free_channel_ids_(new bool[kViEMaxNumberOfChannels]),
|
free_channel_ids_(new bool[kViEMaxNumberOfChannels]),
|
||||||
free_channel_ids_size_(kViEMaxNumberOfChannels),
|
free_channel_ids_size_(kViEMaxNumberOfChannels),
|
||||||
voice_sync_interface_(NULL),
|
voice_sync_interface_(NULL),
|
||||||
voice_engine_(NULL),
|
|
||||||
module_process_thread_(NULL),
|
module_process_thread_(NULL),
|
||||||
engine_config_(config) {
|
engine_config_(config) {
|
||||||
for (int idx = 0; idx < free_channel_ids_size_; idx++) {
|
for (int idx = 0; idx < free_channel_ids_size_; idx++) {
|
||||||
@ -313,7 +312,6 @@ int ViEChannelManager::SetVoiceEngine(VoiceEngine* voice_engine) {
|
|||||||
if (voice_sync_interface_) {
|
if (voice_sync_interface_) {
|
||||||
voice_sync_interface_->Release();
|
voice_sync_interface_->Release();
|
||||||
}
|
}
|
||||||
voice_engine_ = voice_engine;
|
|
||||||
voice_sync_interface_ = sync_interface;
|
voice_sync_interface_ = sync_interface;
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
@ -342,11 +340,6 @@ int ViEChannelManager::DisconnectVoiceChannel(int channel_id) {
|
|||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
VoiceEngine* ViEChannelManager::GetVoiceEngine() {
|
|
||||||
CriticalSectionScoped cs(channel_id_critsect_);
|
|
||||||
return voice_engine_;
|
|
||||||
}
|
|
||||||
|
|
||||||
bool ViEChannelManager::SetRembStatus(int channel_id, bool sender,
|
bool ViEChannelManager::SetRembStatus(int channel_id, bool sender,
|
||||||
bool receiver) {
|
bool receiver) {
|
||||||
CriticalSectionScoped cs(channel_id_critsect_);
|
CriticalSectionScoped cs(channel_id_critsect_);
|
||||||
|
@ -70,8 +70,6 @@ class ViEChannelManager: private ViEManagerBase {
|
|||||||
// Disables lip sync of the channel.
|
// Disables lip sync of the channel.
|
||||||
int DisconnectVoiceChannel(int channel_id);
|
int DisconnectVoiceChannel(int channel_id);
|
||||||
|
|
||||||
VoiceEngine* GetVoiceEngine();
|
|
||||||
|
|
||||||
// Adds a channel to include when sending REMB.
|
// Adds a channel to include when sending REMB.
|
||||||
bool SetRembStatus(int channel_id, bool sender, bool receiver);
|
bool SetRembStatus(int channel_id, bool sender, bool receiver);
|
||||||
|
|
||||||
@ -142,7 +140,6 @@ class ViEChannelManager: private ViEManagerBase {
|
|||||||
EncoderMap vie_encoder_map_;
|
EncoderMap vie_encoder_map_;
|
||||||
VoEVideoSync* voice_sync_interface_;
|
VoEVideoSync* voice_sync_interface_;
|
||||||
|
|
||||||
VoiceEngine* voice_engine_;
|
|
||||||
ProcessThread* module_process_thread_;
|
ProcessThread* module_process_thread_;
|
||||||
const Config& engine_config_;
|
const Config& engine_config_;
|
||||||
};
|
};
|
||||||
|
Loading…
x
Reference in New Issue
Block a user