Small optimisations to audio analysis.
authorCarl Hetherington <cth@carlh.net>
Wed, 26 Aug 2015 21:50:23 +0000 (22:50 +0100)
committerCarl Hetherington <cth@carlh.net>
Wed, 26 Aug 2015 21:50:23 +0000 (22:50 +0100)
src/lib/analyse_audio_job.cc
src/lib/analyse_audio_job.h
src/lib/audio_point.h

index 0660d8f..164c57b 100644 (file)
@@ -43,12 +43,18 @@ AnalyseAudioJob::AnalyseAudioJob (shared_ptr<const Film> film, shared_ptr<const
        , _playlist (playlist)
        , _done (0)
        , _samples_per_point (1)
+       , _current (0)
        , _overall_peak (0)
        , _overall_peak_frame (0)
 {
 
 }
 
+AnalyseAudioJob::~AnalyseAudioJob ()
+{
+       delete[] _current;
+}
+
 string
 AnalyseAudioJob::name () const
 {
@@ -70,7 +76,8 @@ AnalyseAudioJob::run ()
        int64_t const len = _playlist->length().frames_round (_film->audio_frame_rate());
        _samples_per_point = max (int64_t (1), len / _num_points);
 
-       _current.resize (_film->audio_channels ());
+       delete[] _current;
+       _current = new AudioPoint[_film->audio_channels ()];
        _analysis.reset (new AudioAnalysis (_film->audio_channels ()));
 
        bool has_any_audio = false;
@@ -109,19 +116,19 @@ AnalyseAudioJob::run ()
 void
 AnalyseAudioJob::analyse (shared_ptr<const AudioBuffers> b)
 {
-       for (int i = 0; i < b->frames(); ++i) {
-               for (int j = 0; j < b->channels(); ++j) {
+       int const frames = b->frames ();
+       int const channels = b->channels ();
+
+       for (int i = 0; i < frames; ++i) {
+               for (int j = 0; j < channels; ++j) {
                        float s = b->data(j)[i];
-                       if (fabsf (s) < 10e-7) {
+                       float as = fabsf (s);
+                       if (as < 10e-7) {
                                /* SafeStringStream can't serialise and recover inf or -inf, so prevent such
                                   values by replacing with this (140dB down) */
-                               s = 10e-7;
+                               s = as = 10e-7;
                        }
                        _current[j][AudioPoint::RMS] += pow (s, 2);
-                       _current[j][AudioPoint::PEAK] = max (_current[j][AudioPoint::PEAK], fabsf (s));
-
-                       float const as = fabs (s);
-
                        _current[j][AudioPoint::PEAK] = max (_current[j][AudioPoint::PEAK], as);
 
                        if (as > _overall_peak) {
@@ -132,7 +139,6 @@ AnalyseAudioJob::analyse (shared_ptr<const AudioBuffers> b)
                        if ((_done % _samples_per_point) == 0) {
                                _current[j][AudioPoint::RMS] = sqrt (_current[j][AudioPoint::RMS] / _samples_per_point);
                                _analysis->add_point (j, _current[j]);
-
                                _current[j] = AudioPoint ();
                        }
                }
index c255d9c..4273c19 100644 (file)
@@ -39,6 +39,7 @@ class AnalyseAudioJob : public Job
 {
 public:
        AnalyseAudioJob (boost::shared_ptr<const Film>, boost::shared_ptr<const Playlist>);
+       ~AnalyseAudioJob ();
 
        std::string name () const;
        std::string json_name () const;
@@ -51,7 +52,7 @@ private:
 
        int64_t _done;
        int64_t _samples_per_point;
-       std::vector<AudioPoint> _current;
+       AudioPoint* _current;
 
        float _overall_peak;
        Frame _overall_peak_frame;
index 3dfe470..f699233 100644 (file)
@@ -39,7 +39,7 @@ public:
 
        void as_xml (xmlpp::Element *) const;
 
-       float& operator[] (int t) {
+       inline float& operator[] (int t) {
                return _data[t];
        }