Removed dead code.
BUG=C-10062, C-10063, C-10064, C-10065, C-10393, C-10394. Review URL: https://webrtc-codereview.appspot.com/343013 git-svn-id: http://webrtc.googlecode.com/svn/trunk@1448 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
parent
7f2c2a5db2
commit
a768ca13f4
@ -301,10 +301,6 @@ int ViEChannelManager::SetVoiceEngine(VoiceEngine* voice_engine) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, ViEId(engine_id_),
|
||||
"%s Can't get audio sync interface from VoiceEngine.",
|
||||
__FUNCTION__);
|
||||
|
||||
if (sync_interface) {
|
||||
sync_interface->Release();
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
|
||||
@ -531,47 +531,35 @@ int ViEFileImpl::GetRenderSnapshot(const int video_channel,
|
||||
return -1;
|
||||
}
|
||||
|
||||
const int JPEG_FORMAT = 0;
|
||||
int format = JPEG_FORMAT;
|
||||
switch (format) {
|
||||
case JPEG_FORMAT: {
|
||||
// JPEGEncoder writes the jpeg file for you (no control over it) and does
|
||||
// not return you the buffer. Thus, we are not going to be writing to the
|
||||
// disk here.
|
||||
JpegEncoder jpeg_encoder;
|
||||
RawImage input_image;
|
||||
if (jpeg_encoder.SetFileName(file_nameUTF8) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not open output file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
return -1;
|
||||
}
|
||||
// JPEGEncoder writes the jpeg file for you (no control over it) and does
|
||||
// not return you the buffer. Thus, we are not going to be writing to the
|
||||
// disk here.
|
||||
JpegEncoder jpeg_encoder;
|
||||
RawImage input_image;
|
||||
if (jpeg_encoder.SetFileName(file_nameUTF8) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not open output file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
return -1;
|
||||
}
|
||||
|
||||
input_image._width = video_frame.Width();
|
||||
input_image._height = video_frame.Height();
|
||||
video_frame.Swap(input_image._buffer, input_image._length,
|
||||
input_image._size);
|
||||
input_image._width = video_frame.Width();
|
||||
input_image._height = video_frame.Height();
|
||||
video_frame.Swap(input_image._buffer, input_image._length,
|
||||
input_image._size);
|
||||
|
||||
if (jpeg_encoder.Encode(input_image) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not encode i420 -> jpeg file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
if (jpeg_encoder.Encode(input_image) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not encode i420 -> jpeg file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
input_image._buffer = NULL;
|
||||
break;
|
||||
}
|
||||
default: {
|
||||
WEBRTC_TRACE(kTraceError, kTraceFile, instance_id_,
|
||||
"\tUnsupported file format for %s", __FUNCTION__);
|
||||
return -1;
|
||||
break;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
delete [] input_image._buffer;
|
||||
input_image._buffer = NULL;
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -619,50 +607,37 @@ int ViEFileImpl::GetCaptureDeviceSnapshot(const int capture_id,
|
||||
return -1;
|
||||
}
|
||||
|
||||
const int JPEG_FORMAT = 0;
|
||||
int format = JPEG_FORMAT;
|
||||
switch (format) {
|
||||
case JPEG_FORMAT: {
|
||||
// JPEGEncoder writes the jpeg file for you (no control over it) and does
|
||||
// not return you the buffer Thusly, we are not going to be writing to the
|
||||
// disk here.
|
||||
JpegEncoder jpeg_encoder;
|
||||
RawImage input_image;
|
||||
input_image._width = video_frame.Width();
|
||||
input_image._height = video_frame.Height();
|
||||
video_frame.Swap(input_image._buffer, input_image._length,
|
||||
input_image._size);
|
||||
// JPEGEncoder writes the jpeg file for you (no control over it) and does
|
||||
// not return you the buffer Thusly, we are not going to be writing to the
|
||||
// disk here.
|
||||
JpegEncoder jpeg_encoder;
|
||||
RawImage input_image;
|
||||
input_image._width = video_frame.Width();
|
||||
input_image._height = video_frame.Height();
|
||||
video_frame.Swap(input_image._buffer, input_image._length,
|
||||
input_image._size);
|
||||
|
||||
if (jpeg_encoder.SetFileName(file_nameUTF8) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not open output file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
if (jpeg_encoder.SetFileName(file_nameUTF8) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not open output file '%s' for writing!",
|
||||
file_nameUTF8);
|
||||
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
if (jpeg_encoder.Encode(input_image) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not encode i420 -> jpeg file '%s' for "
|
||||
"writing!", file_nameUTF8);
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
input_image._buffer = NULL;
|
||||
break;
|
||||
}
|
||||
default: {
|
||||
WEBRTC_TRACE(kTraceError, kTraceFile, instance_id_,
|
||||
"\tUnsupported file format for %s", __FUNCTION__);
|
||||
return -1;
|
||||
break;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
if (jpeg_encoder.Encode(input_image) == -1) {
|
||||
WEBRTC_TRACE(kTraceError, kTraceVideo, instance_id_,
|
||||
"\tCould not encode i420 -> jpeg file '%s' for "
|
||||
"writing!", file_nameUTF8);
|
||||
if (input_image._buffer) {
|
||||
delete [] input_image._buffer;
|
||||
}
|
||||
return -1;
|
||||
}
|
||||
delete [] input_image._buffer;
|
||||
input_image._buffer = NULL;
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
@ -88,6 +88,7 @@ int ViEFileRecorder::StartRecording(const char* file_nameUTF8,
|
||||
audio_codec_inst.plfreq * 16 };
|
||||
|
||||
switch (audio_source) {
|
||||
// case NO_AUDIO is checked above.
|
||||
case MICROPHONE:
|
||||
error = voe_file_interface_->StartRecordingMicrophone(
|
||||
this, &engine_audio_codec_inst);
|
||||
@ -96,8 +97,6 @@ int ViEFileRecorder::StartRecording(const char* file_nameUTF8,
|
||||
error = voe_file_interface_->StartRecordingPlayout(
|
||||
audio_channel, this, &engine_audio_codec_inst);
|
||||
break;
|
||||
case NO_AUDIO:
|
||||
break;
|
||||
default:
|
||||
assert(!"Unknown audio_source");
|
||||
}
|
||||
|
||||
@ -158,8 +158,6 @@ int ViERenderImpl::AddRenderer(const int render_id, void* window,
|
||||
}
|
||||
return frame_provider->RegisterFrameCallback(render_id, renderer);
|
||||
}
|
||||
SetLastError(kViERenderInvalidRenderId);
|
||||
return -1;
|
||||
}
|
||||
|
||||
int ViERenderImpl::RemoveRenderer(const int render_id) {
|
||||
@ -386,8 +384,6 @@ int ViERenderImpl::AddRenderer(const int render_id,
|
||||
}
|
||||
return frame_provider->RegisterFrameCallback(render_id, renderer);
|
||||
}
|
||||
SetLastError(kViERenderInvalidRenderId);
|
||||
return -1;
|
||||
}
|
||||
|
||||
} // namespace webrtc
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user