Chromium Code Reviews| Index: media/tools/media_bench/media_bench.cc |
| =================================================================== |
| --- media/tools/media_bench/media_bench.cc (revision 90887) |
| +++ media/tools/media_bench/media_bench.cc (working copy) |
| @@ -132,15 +132,15 @@ |
| FilePath out_path; |
| if (filenames.size() > 1) |
| out_path = FilePath(filenames[1]); |
| - CodecType target_codec = CODEC_TYPE_UNKNOWN; |
| + AVMediaType target_codec = AVMEDIA_TYPE_UNKNOWN; |
| // Determine whether to benchmark audio or video decoding. |
| std::string stream(cmd_line->GetSwitchValueASCII(switches::kStream)); |
| if (!stream.empty()) { |
| if (stream.compare("audio") == 0) { |
| - target_codec = CODEC_TYPE_AUDIO; |
| + target_codec = AVMEDIA_TYPE_AUDIO; |
| } else if (stream.compare("video") == 0) { |
| - target_codec = CODEC_TYPE_VIDEO; |
| + target_codec = AVMEDIA_TYPE_VIDEO; |
| } else { |
| std::cerr << "Unknown --stream option " << stream << std::endl; |
| return 1; |
| @@ -239,7 +239,7 @@ |
| NULL, 0, NULL); |
| if (result < 0) { |
| switch (result) { |
| - case AVERROR_NOFMT: |
| + case AVERROR(EINVAL): |
| std::cerr << "Error: File format not supported " |
| << in_path.value() << std::endl; |
| break; |
| @@ -293,7 +293,7 @@ |
| *log_out << " "; |
| } |
| - if (!codec || (codec_context->codec_type == CODEC_TYPE_UNKNOWN)) { |
| + if (!codec || (codec_context->codec_type == AVMEDIA_TYPE_UNKNOWN)) { |
| *log_out << "Stream #" << i << ": Unknown" << std::endl; |
| } else { |
| // Print out stream information |
| @@ -339,11 +339,9 @@ |
| } |
| // Initialize threaded decode. |
| - if (target_codec == CODEC_TYPE_VIDEO && video_threads > 0) { |
| - if (avcodec_thread_init(codec_context, video_threads) < 0) { |
| - std::cerr << "Warning: Could not initialize threading!\n" |
| - << "Did you build with pthread/w32thread support?" << std::endl; |
| - } |
| + if (target_codec == AVMEDIA_TYPE_VIDEO && video_threads > 0) { |
| + codec_context->thread_count = video_threads; |
| + // avcodec_thread_init deprecated |
|
scherkus (not reviewing)
2011/06/29 17:00:36
ditto
ilja
2011/06/29 21:40:05
Done.
|
| } |
| // Initialize our codec. |
| @@ -402,7 +400,7 @@ |
| // Only decode packets from our target stream. |
| if (packet.stream_index == target_stream) { |
| int result = -1; |
| - if (target_codec == CODEC_TYPE_AUDIO) { |
| + if (target_codec == AVMEDIA_TYPE_AUDIO) { |
| int size_out = AVCODEC_MAX_AUDIO_FRAME_SIZE; |
| base::TimeTicks decode_start = base::TimeTicks::HighResNow(); |
| @@ -434,7 +432,7 @@ |
| MD5Update(&ctx, u8_samples, size_out); |
| } |
| } |
| - } else if (target_codec == CODEC_TYPE_VIDEO) { |
| + } else if (target_codec == AVMEDIA_TYPE_VIDEO) { |
| int got_picture = 0; |
| base::TimeTicks decode_start = base::TimeTicks::HighResNow(); |