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

Unified Diff: webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc

Issue 2685783014: Replace NULL with nullptr in all C++ files. (Closed)
Patch Set: Fixing android. Created 3 years, 10 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/video_coding/codecs/vp9/vp9_impl.cc
diff --git a/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc b/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc
index 82afd9e6c0fa6437910d55e17d59d39a5074bfc5..8af0bf04a21392861998bf5f87354bb6b13d3c8d 100644
--- a/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc
+++ b/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc
@@ -64,16 +64,16 @@ void VP9EncoderImpl::EncoderOutputCodedPacketCallback(vpx_codec_cx_pkt* pkt,
VP9EncoderImpl::VP9EncoderImpl()
: encoded_image_(),
- encoded_complete_callback_(NULL),
+ encoded_complete_callback_(nullptr),
inited_(false),
timestamp_(0),
picture_id_(0),
cpu_speed_(3),
rc_max_intra_target_(0),
- encoder_(NULL),
- config_(NULL),
- raw_(NULL),
- input_image_(NULL),
+ encoder_(nullptr),
+ config_(nullptr),
+ raw_(nullptr),
+ input_image_(nullptr),
tl0_pic_idx_(0),
frames_since_kf_(0),
num_temporal_layers_(0),
@@ -93,24 +93,24 @@ VP9EncoderImpl::~VP9EncoderImpl() {
}
int VP9EncoderImpl::Release() {
- if (encoded_image_._buffer != NULL) {
+ if (encoded_image_._buffer != nullptr) {
delete[] encoded_image_._buffer;
- encoded_image_._buffer = NULL;
+ encoded_image_._buffer = nullptr;
}
- if (encoder_ != NULL) {
+ if (encoder_ != nullptr) {
if (vpx_codec_destroy(encoder_)) {
return WEBRTC_VIDEO_CODEC_MEMORY;
}
delete encoder_;
- encoder_ = NULL;
+ encoder_ = nullptr;
}
- if (config_ != NULL) {
+ if (config_ != nullptr) {
delete config_;
- config_ = NULL;
+ config_ = nullptr;
}
- if (raw_ != NULL) {
+ if (raw_ != nullptr) {
vpx_img_free(raw_);
- raw_ = NULL;
+ raw_ = nullptr;
}
inited_ = false;
return WEBRTC_VIDEO_CODEC_OK;
@@ -232,7 +232,7 @@ int VP9EncoderImpl::SetRateAllocation(
int VP9EncoderImpl::InitEncode(const VideoCodec* inst,
int number_of_cores,
size_t /*max_payload_size*/) {
- if (inst == NULL) {
+ if (inst == nullptr) {
return WEBRTC_VIDEO_CODEC_ERR_PARAMETER;
}
if (inst->maxFramerate < 1) {
@@ -260,14 +260,14 @@ int VP9EncoderImpl::InitEncode(const VideoCodec* inst,
if (ret_val < 0) {
return ret_val;
}
- if (encoder_ == NULL) {
+ if (encoder_ == nullptr) {
encoder_ = new vpx_codec_ctx_t;
// Only randomize pid/tl0 the first time the encoder is initialized
// in order to not make random jumps mid-stream.
picture_id_ = static_cast<uint16_t>(rand()) & 0x7FFF; // NOLINT
tl0_pic_idx_ = static_cast<uint8_t>(rand()); // NOLINT
}
- if (config_ == NULL) {
+ if (config_ == nullptr) {
config_ = new vpx_codec_enc_cfg_t;
}
timestamp_ = 0;
@@ -281,17 +281,17 @@ int VP9EncoderImpl::InitEncode(const VideoCodec* inst,
num_temporal_layers_ = 1;
// Allocate memory for encoded image
- if (encoded_image_._buffer != NULL) {
+ if (encoded_image_._buffer != nullptr) {
delete[] encoded_image_._buffer;
}
encoded_image_._size = CalcBufferSize(kI420, codec_.width, codec_.height);
encoded_image_._buffer = new uint8_t[encoded_image_._size];
encoded_image_._completeFrame = true;
- // Creating a wrapper to the image - setting image data to NULL. Actual
+ // Creating a wrapper to the image - setting image data to null. Actual
// pointer will be set in encode. Setting align to 1, as it is meaningless
// (actual memory is not allocated).
- raw_ = vpx_img_wrap(NULL, VPX_IMG_FMT_I420, codec_.width, codec_.height, 1,
- NULL);
+ raw_ = vpx_img_wrap(nullptr, VPX_IMG_FMT_I420, codec_.width, codec_.height, 1,
+ nullptr);
// Populate encoder configuration with default values.
if (vpx_codec_enc_config_default(vpx_codec_vp9_cx(), config_, 0)) {
return WEBRTC_VIDEO_CODEC_ERROR;
@@ -487,7 +487,7 @@ int VP9EncoderImpl::Encode(const VideoFrame& input_image,
if (!inited_) {
return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
}
- if (encoded_complete_callback_ == NULL) {
+ if (encoded_complete_callback_ == nullptr) {
return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
}
FrameType frame_type = kVideoFrameDelta;
@@ -559,7 +559,7 @@ int VP9EncoderImpl::Encode(const VideoFrame& input_image,
void VP9EncoderImpl::PopulateCodecSpecific(CodecSpecificInfo* codec_specific,
const vpx_codec_cx_pkt& pkt,
uint32_t timestamp) {
- assert(codec_specific != NULL);
+ assert(codec_specific != nullptr);
codec_specific->codecType = kVideoCodecVP9;
codec_specific->codec_name = ImplementationName();
CodecSpecificInfoVP9* vp9_info = &(codec_specific->codecSpecific.VP9);
@@ -839,9 +839,9 @@ VP9Decoder* VP9Decoder::Create() {
}
VP9DecoderImpl::VP9DecoderImpl()
- : decode_complete_callback_(NULL),
+ : decode_complete_callback_(nullptr),
inited_(false),
- decoder_(NULL),
+ decoder_(nullptr),
key_frame_required_(true) {
memset(&codec_, 0, sizeof(codec_));
}
@@ -860,14 +860,14 @@ VP9DecoderImpl::~VP9DecoderImpl() {
}
int VP9DecoderImpl::InitDecode(const VideoCodec* inst, int number_of_cores) {
- if (inst == NULL) {
+ if (inst == nullptr) {
return WEBRTC_VIDEO_CODEC_ERR_PARAMETER;
}
int ret_val = Release();
if (ret_val < 0) {
return ret_val;
}
- if (decoder_ == NULL) {
+ if (decoder_ == nullptr) {
decoder_ = new vpx_codec_ctx_t;
}
vpx_codec_dec_cfg_t cfg;
@@ -901,7 +901,7 @@ int VP9DecoderImpl::Decode(const EncodedImage& input_image,
if (!inited_) {
return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
}
- if (decode_complete_callback_ == NULL) {
+ if (decode_complete_callback_ == nullptr) {
return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
}
// Always start with a complete key frame.
@@ -915,11 +915,11 @@ int VP9DecoderImpl::Decode(const EncodedImage& input_image,
return WEBRTC_VIDEO_CODEC_ERROR;
}
}
- vpx_codec_iter_t iter = NULL;
+ vpx_codec_iter_t iter = nullptr;
vpx_image_t* img;
uint8_t* buffer = input_image._buffer;
if (input_image._length == 0) {
- buffer = NULL; // Triggers full frame concealment.
+ buffer = nullptr; // Triggers full frame concealment.
}
// During decode libvpx may get and release buffers from |frame_buffer_pool_|.
// In practice libvpx keeps a few (~3-4) buffers alive at a time.
@@ -942,8 +942,8 @@ int VP9DecoderImpl::Decode(const EncodedImage& input_image,
int VP9DecoderImpl::ReturnFrame(const vpx_image_t* img,
uint32_t timestamp,
int64_t ntp_time_ms) {
- if (img == NULL) {
- // Decoder OK and NULL image => No show frame.
+ if (img == nullptr) {
+ // Decoder OK and null image => No show frame.
return WEBRTC_VIDEO_CODEC_NO_OUTPUT;
}
@@ -982,14 +982,14 @@ int VP9DecoderImpl::RegisterDecodeCompleteCallback(
}
int VP9DecoderImpl::Release() {
- if (decoder_ != NULL) {
+ if (decoder_ != nullptr) {
// When a codec is destroyed libvpx will release any buffers of
// |frame_buffer_pool_| it is currently using.
if (vpx_codec_destroy(decoder_)) {
return WEBRTC_VIDEO_CODEC_MEMORY;
}
delete decoder_;
- decoder_ = NULL;
+ decoder_ = nullptr;
}
// Releases buffers from the pool. Any buffers not in use are deleted. Buffers
// still referenced externally are deleted once fully released, not returning

Powered by Google App Engine
This is Rietveld 408576698