frames: add log to warn about possible data already allocated

Clement Champetier · Clement Champetier · commit cd2db7b94a29 · 2016-11-03T13:56:52.000+01:00
diff --git a/src/AvTranscoder/data/decoded/AudioFrame.cpp b/src/AvTranscoder/data/decoded/AudioFrame.cpp
@@ -98,6 +98,9 @@ size_t AudioFrame::getDataSize() const
 
 void AudioFrame::allocateData()
 {
+    if(_dataAllocated)
+        LOG_WARN("The AudioFrame seems to already have allocated data. This could lead to memory leaks.")
+
     // Set Frame properties
     av_frame_set_sample_rate(_frame, _desc._sampleRate);
     av_frame_set_channels(_frame, _desc._nbChannels);
diff --git a/src/AvTranscoder/data/decoded/VideoFrame.cpp b/src/AvTranscoder/data/decoded/VideoFrame.cpp
@@ -83,6 +83,9 @@ size_t VideoFrame::getDataSize() const
 
 void VideoFrame::allocateData()
 {
+    if(_dataAllocated)
+        LOG_WARN("The VideoFrame seems to already have allocated data. This could lead to memory leaks.")
+
     // Set Frame properties
     _frame->width = _desc._width;
     _frame->height = _desc._height;

Original file line number	Diff line number	Diff line change
`@@ -98,6 +98,9 @@ size_t AudioFrame::getDataSize() const`
`98`	`98`
`99`	`99`	`void AudioFrame::allocateData()`
`100`	`100`	`{`
	`101`	`+ if(_dataAllocated)`
	`102`	`+ LOG_WARN("The AudioFrame seems to already have allocated data. This could lead to memory leaks.")`
	`103`	`+`
`101`	`104`	`// Set Frame properties`
`102`	`105`	`av_frame_set_sample_rate(_frame, _desc._sampleRate);`
`103`	`106`	`av_frame_set_channels(_frame, _desc._nbChannels);`
Original file line number	Diff line number	Diff line change
`@@ -83,6 +83,9 @@ size_t VideoFrame::getDataSize() const`
`83`	`83`
`84`	`84`	`void VideoFrame::allocateData()`
`85`	`85`	`{`
	`86`	`+ if(_dataAllocated)`
	`87`	`+ LOG_WARN("The VideoFrame seems to already have allocated data. This could lead to memory leaks.")`
	`88`	`+`
`86`	`89`	`// Set Frame properties`
`87`	`90`	`_frame->width = _desc._width;`
`88`	`91`	`_frame->height = _desc._height;`