Use make_shared<>.
[dcpomatic.git] / src / lib / audio_buffers.cc
index 56ca7a94b18c5860e3800f03740baa2c09480d59..4c3c2d4a7a20515b8f58d4acbf17d1feb18b2039 100644 (file)
@@ -1,36 +1,40 @@
 /*
     Copyright (C) 2012-2013 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
 
-    This program is distributed in the hope that it will be useful,
+    DCP-o-matic is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
     You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
 
 */
 
+#include "audio_buffers.h"
+#include "dcpomatic_assert.h"
+#include <boost/make_shared.hpp>
 #include <cassert>
 #include <cstring>
 #include <cmath>
 #include <stdexcept>
-#include "audio_buffers.h"
 
 using std::bad_alloc;
 using boost::shared_ptr;
+using boost::make_shared;
 
 /** Construct an AudioBuffers.  Audio data is undefined after this constructor.
  *  @param channels Number of channels.
  *  @param frames Number of frames to reserve space for.
  */
-AudioBuffers::AudioBuffers (int channels, int frames)
+AudioBuffers::AudioBuffers (int channels, int32_t frames)
 {
        allocate (channels, frames);
 }
@@ -56,7 +60,7 @@ AudioBuffers::operator= (AudioBuffers const & other)
        if (this == &other) {
                return *this;
        }
-               
+
        deallocate ();
        allocate (other._channels, other._frames);
        copy_from (&other, other._frames, 0, 0);
@@ -71,20 +75,20 @@ AudioBuffers::~AudioBuffers ()
 }
 
 void
-AudioBuffers::allocate (int channels, int frames)
+AudioBuffers::allocate (int channels, int32_t frames)
 {
-       assert (frames >= 0);
-       assert (channels >= 0);
+       DCPOMATIC_ASSERT (frames >= 0);
+       DCPOMATIC_ASSERT (channels >= 0);
 
        _channels = channels;
        _frames = frames;
        _allocated_frames = frames;
-       
+
        _data = static_cast<float**> (malloc (_channels * sizeof (float *)));
        if (!_data) {
                throw bad_alloc ();
        }
-       
+
        for (int i = 0; i < _channels; ++i) {
                _data[i] = static_cast<float*> (malloc (frames * sizeof (float)));
                if (!_data[i]) {
@@ -109,7 +113,7 @@ AudioBuffers::deallocate ()
 float*
 AudioBuffers::data (int c) const
 {
-       assert (c >= 0 && c < _channels);
+       DCPOMATIC_ASSERT (c >= 0 && c < _channels);
        return _data[c];
 }
 
@@ -119,16 +123,16 @@ AudioBuffers::data (int c) const
  *  @param f Frames; must be less than or equal to the number of allocated frames.
  */
 void
-AudioBuffers::set_frames (int f)
+AudioBuffers::set_frames (int32_t f)
 {
-       assert (f <= _allocated_frames);
+       DCPOMATIC_ASSERT (f <= _allocated_frames);
 
        for (int c = 0; c < _channels; ++c) {
                for (int i = f; i < _frames; ++i) {
                        _data[c][i] = 0;
                }
        }
-       
+
        _frames = f;
 }
 
@@ -147,17 +151,17 @@ AudioBuffers::make_silent ()
 void
 AudioBuffers::make_silent (int c)
 {
-       assert (c >= 0 && c < _channels);
-       
+       DCPOMATIC_ASSERT (c >= 0 && c < _channels);
+
        for (int i = 0; i < _frames; ++i) {
                _data[c][i] = 0;
        }
 }
 
 void
-AudioBuffers::make_silent (int from, int frames)
+AudioBuffers::make_silent (int32_t from, int32_t frames)
 {
-       assert ((from + frames) <= _allocated_frames);
+       DCPOMATIC_ASSERT ((from + frames) <= _allocated_frames);
 
        for (int c = 0; c < _channels; ++c) {
                for (int i = from; i < (from + frames); ++i) {
@@ -173,18 +177,18 @@ AudioBuffers::make_silent (int from, int frames)
  *  @param write_offset Offset to write to in `to'.
  */
 void
-AudioBuffers::copy_from (AudioBuffers const * from, int frames_to_copy, int read_offset, int write_offset)
+AudioBuffers::copy_from (AudioBuffers const * from, int32_t frames_to_copy, int32_t read_offset, int32_t write_offset)
 {
        if (frames_to_copy == 0) {
                /* Prevent the asserts from firing if there is nothing to do */
                return;
        }
-       
-       assert (from->channels() == channels());
 
-       assert (from);
-       assert (read_offset >= 0 && (read_offset + frames_to_copy) <= from->_allocated_frames);
-       assert (write_offset >= 0 && (write_offset + frames_to_copy) <= _allocated_frames);
+       DCPOMATIC_ASSERT (from->channels() == channels());
+
+       DCPOMATIC_ASSERT (from);
+       DCPOMATIC_ASSERT (read_offset >= 0 && (read_offset + frames_to_copy) <= from->_allocated_frames);
+       DCPOMATIC_ASSERT (write_offset >= 0 && (write_offset + frames_to_copy) <= _allocated_frames);
 
        for (int i = 0; i < _channels; ++i) {
                memcpy (_data[i] + write_offset, from->_data[i] + read_offset, frames_to_copy * sizeof(float));
@@ -196,23 +200,23 @@ AudioBuffers::copy_from (AudioBuffers const * from, int frames_to_copy, int read
  *  @param to Offset to move to.
  *  @param frames Number of frames to move.
  */
-    
+
 void
-AudioBuffers::move (int from, int to, int frames)
+AudioBuffers::move (int32_t from, int32_t to, int32_t frames)
 {
        if (frames == 0) {
                return;
        }
-       
-       assert (from >= 0);
-       assert (from < _frames);
-       assert (to >= 0);
-       assert (to < _frames);
-       assert (frames > 0);
-       assert (frames <= _frames);
-       assert ((from + frames) <= _frames);
-       assert ((to + frames) <= _allocated_frames);
-       
+
+       DCPOMATIC_ASSERT (from >= 0);
+       DCPOMATIC_ASSERT (from < _frames);
+       DCPOMATIC_ASSERT (to >= 0);
+       DCPOMATIC_ASSERT (to < _frames);
+       DCPOMATIC_ASSERT (frames > 0);
+       DCPOMATIC_ASSERT (frames <= _frames);
+       DCPOMATIC_ASSERT ((from + frames) <= _frames);
+       DCPOMATIC_ASSERT ((to + frames) <= _allocated_frames);
+
        for (int i = 0; i < _channels; ++i) {
                memmove (_data[i] + to, _data[i] + from, frames * sizeof(float));
        }
@@ -225,8 +229,8 @@ void
 AudioBuffers::accumulate_channel (AudioBuffers const * from, int from_channel, int to_channel, float gain)
 {
        int const N = frames ();
-       assert (from->frames() == N);
-       assert (to_channel <= _channels);
+       DCPOMATIC_ASSERT (from->frames() == N);
+       DCPOMATIC_ASSERT (to_channel <= _channels);
 
        float* s = from->data (from_channel);
        float* d = _data[to_channel];
@@ -240,12 +244,23 @@ AudioBuffers::accumulate_channel (AudioBuffers const * from, int from_channel, i
  *  the buffers, fill the new space with silence.
  */
 void
-AudioBuffers::ensure_size (int frames)
+AudioBuffers::ensure_size (int32_t frames)
 {
        if (_allocated_frames >= frames) {
                return;
        }
 
+       /* Round up frames to the next power of 2 to reduce the number
+          of realloc()s that are necessary.
+       */
+       frames--;
+       frames |= frames >> 1;
+       frames |= frames >> 2;
+       frames |= frames >> 4;
+       frames |= frames >> 8;
+       frames |= frames >> 16;
+       frames++;
+
        for (int i = 0; i < _channels; ++i) {
                _data[i] = static_cast<float*> (realloc (_data[i], frames * sizeof (float)));
                if (!_data[i]) {
@@ -260,11 +275,11 @@ AudioBuffers::ensure_size (int frames)
 }
 
 void
-AudioBuffers::accumulate_frames (AudioBuffers const * from, int read_offset, int write_offset, int frames)
+AudioBuffers::accumulate_frames (AudioBuffers const * from, int32_t read_offset, int32_t write_offset, int32_t frames)
 {
-       assert (_channels == from->channels ());
-       assert (read_offset >= 0);
-       assert (write_offset >= 0);
+       DCPOMATIC_ASSERT (_channels == from->channels ());
+       DCPOMATIC_ASSERT (read_offset >= 0);
+       DCPOMATIC_ASSERT (write_offset >= 0);
 
        for (int i = 0; i < _channels; ++i) {
                for (int j = 0; j < frames; ++j) {
@@ -278,7 +293,7 @@ void
 AudioBuffers::apply_gain (float dB)
 {
        float const linear = pow (10, dB / 20);
-       
+
        for (int i = 0; i < _channels; ++i) {
                for (int j = 0; j < _frames; ++j) {
                        _data[i][j] *= linear;
@@ -292,7 +307,7 @@ AudioBuffers::apply_gain (float dB)
 shared_ptr<AudioBuffers>
 AudioBuffers::channel (int c) const
 {
-       shared_ptr<AudioBuffers> o (new AudioBuffers (1, frames ()));
+       shared_ptr<AudioBuffers> o = make_shared<AudioBuffers> (1, frames ());
        o->copy_channel_from (this, c, 0);
        return o;
 }
@@ -300,14 +315,14 @@ AudioBuffers::channel (int c) const
 void
 AudioBuffers::copy_channel_from (AudioBuffers const * from, int from_channel, int to_channel)
 {
-       assert (from->frames() == frames());
+       DCPOMATIC_ASSERT (from->frames() == frames());
        memcpy (data(to_channel), from->data(from_channel), frames() * sizeof (float));
 }
 
 shared_ptr<AudioBuffers>
 AudioBuffers::clone () const
 {
-       shared_ptr<AudioBuffers> b (new AudioBuffers (channels (), frames ()));
+       shared_ptr<AudioBuffers> b = make_shared<AudioBuffers> (channels (), frames ());
        b->copy_from (this, frames (), 0, 0);
        return b;
 }