pipewire: cleanup

git-svn-id: http://svn.code.sf.net/p/qmmp-dev/code/trunk/qmmp@9846 90c681e8-e032-0410-971d-27865f9a5e38

2 files changed, 52 insertions, 82 deletions

diff --git a/src/plugins/Output/pipewire/outputpipewire.cpp b/src/plugins/Output/pipewire/outputpipewire.cpp
index c1304afea..75d05d856 100644
--- a/src/plugins/Output/pipewire/outputpipewire.cpp
+++ b/src/plugins/Output/pipewire/outputpipewire.cpp
@@ -56,34 +56,25 @@ OutputPipeWire::~OutputPipeWire()
 }
 
 void OutputPipeWire::onStateChanged (void *data, enum pw_stream_state old,
-                       enum pw_stream_state state, const char *error)
+                                     enum pw_stream_state state, const char *error)
 {
+    Q_UNUSED(old);
+    Q_UNUSED(error);
+
     OutputPipeWire *o = static_cast<OutputPipeWire *>(data);
 
-    if (o->m_ignoreStateChange)
+    if(o->m_ignoreStateChange)
         return;
 
-    switch (state)
+    if(state == PW_STREAM_STATE_UNCONNECTED ||
+            state == PW_STREAM_STATE_PAUSED ||
+            state == PW_STREAM_STATE_STREAMING)
     {
-    case PW_STREAM_STATE_UNCONNECTED:
-        pw_thread_loop_signal(o->m_loop, false);
-        break;
-    case PW_STREAM_STATE_PAUSED:
-        o->m_streamPaused = true;
-        qDebug("paused");
         pw_thread_loop_signal(o->m_loop, false);
-        break;
-    case PW_STREAM_STATE_STREAMING:
-        o->m_streamPaused = false;
-        pw_thread_loop_signal(o->m_loop, false);
-        if(volumeControl)
+
+        if(state == PW_STREAM_STATE_STREAMING && volumeControl)
             instance->setMuted(volumeControl->isMuted());
-        break;
-    default:
-        break;
     }
-
-    //pw_thread_loop_signal(o->m_loop, false);
 }
 
 bool OutputPipeWire::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFormat format)
@@ -133,38 +124,37 @@ bool OutputPipeWire::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFormat
 
     m_coreInitSeq = pw_core_sync(m_core, PW_ID_CORE, m_coreInitSeq);
 
-
-    if (pw_thread_loop_start(m_loop) != 0)
+    if(pw_thread_loop_start(m_loop) != 0)
     {
-        //m_err = true;
+        qWarning("OutputPipeWire: unable to start loop");
         return false;
     }
 
     pw_thread_loop_lock(m_loop);
-    while (!m_initDone)
+    while(!m_inited)
     {
-        if (pw_thread_loop_timed_wait(m_loop, 2) != 0)
+        if(pw_thread_loop_timed_wait(m_loop, 2) != 0)
             break;
     }
     pw_thread_loop_unlock(m_loop);
 
-    if(!m_initDone || !m_hasSinks)
+    if(!m_inited || !m_hasSinks)
+    {
+        qWarning("OutputPipeWire: unable to initialize loop");
         return false;
+    }
 
     m_stride = AudioParameters::sampleSize(format) * map.count();
-    m_nFrames = qBound(64, qCeil((2048.0 / 48000.0) * freq), 8192);
-    m_bufferSize = m_nFrames * m_stride;
-    m_writeBufferPos = 0;
+    m_frames = qBound(64, qCeil(2048.0 * freq / 48000.0), 8192);
+    m_bufferSize = m_frames * m_stride;
     m_buffer = new unsigned char[m_bufferSize];
 
-    auto props = pw_properties_new(
-                PW_KEY_MEDIA_TYPE, "Audio",
-                PW_KEY_MEDIA_CATEGORY, "Playback",
-                PW_KEY_MEDIA_ROLE, "Music",
-                PW_KEY_MEDIA_ICON_NAME, "qmmp",
-                nullptr
-                );
-    pw_properties_setf(props, PW_KEY_NODE_LATENCY, "%u/%u", m_nFrames, freq);
+    pw_properties *props = pw_properties_new(PW_KEY_MEDIA_TYPE, "Audio",
+                                             PW_KEY_MEDIA_CATEGORY, "Playback",
+                                             PW_KEY_MEDIA_ROLE, "Music",
+                                             PW_KEY_MEDIA_ICON_NAME, "qmmp",
+                                             nullptr);
+    pw_properties_setf(props, PW_KEY_NODE_LATENCY, "%u/%u", m_frames, freq);
 
     pw_thread_loop_lock(m_loop);
 
@@ -188,7 +178,7 @@ bool OutputPipeWire::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFormat
         .format = SPA_AUDIO_FORMAT_F32,
         .flags = SPA_AUDIO_FLAG_NONE,
         .rate = freq,
-        .channels = map.count()
+        .channels = static_cast<unsigned int>(map.count())
     };
 
     switch (format)
@@ -229,16 +219,16 @@ bool OutputPipeWire::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFormat
 
 
 
-
-    qDebug() << pw_stream_connect(m_stream,
-                                  PW_DIRECTION_OUTPUT,
-                                  PW_ID_ANY,
-                                  static_cast<pw_stream_flags>(PW_STREAM_FLAG_AUTOCONNECT |
+    pw_stream_flags streamFlags = static_cast<pw_stream_flags>(PW_STREAM_FLAG_AUTOCONNECT |
                                                                PW_STREAM_FLAG_MAP_BUFFERS |
-                                                               PW_STREAM_FLAG_RT_PROCESS),
-                      params, 1);
-
+                                                               PW_STREAM_FLAG_RT_PROCESS);
 
+    if(pw_stream_connect(m_stream, PW_DIRECTION_OUTPUT, PW_ID_ANY, streamFlags, params, 1) != 0)
+    {
+        pw_thread_loop_unlock(m_loop);
+        qDebug("OutputPipeWire: unable to connect stream");
+        return false;
+    }
 
     Output::configure(freq, map, format);
     pw_thread_loop_unlock(m_loop);
@@ -248,17 +238,11 @@ bool OutputPipeWire::initialize(quint32 freq, ChannelMap map, Qmmp::AudioFormat
 
 qint64 OutputPipeWire::latency()
 {
-    /*pa_usec_t usec;
-    int negative;
-
-    int r = pa_stream_get_latency(m_stream, &usec, &negative);
-    return (r == PA_OK && negative == 0) ? (usec / 1000) : 0;*/
-    return 0;
+    return (m_buffer_at / m_stride + m_frames) * 1000 / sampleRate();
 }
 
 qint64 OutputPipeWire::writeAudio(unsigned char *data, qint64 maxSize)
 {
-    //qDebug() << Q_FUNC_INFO;
     pw_thread_loop_lock(m_loop);
 
     if(m_buffer_at == m_bufferSize)
@@ -317,7 +301,6 @@ void OutputPipeWire::setMuted(bool mute)
 
 void OutputPipeWire::setVolume(const VolumeSettings &v)
 {
-    qDebug() << Q_FUNC_INFO << v.left << v.right;
     pw_thread_loop_lock(m_loop);
 
     float* values = new float[channels()];
@@ -334,7 +317,7 @@ void OutputPipeWire::setVolume(const VolumeSettings &v)
     }
 
     pw_stream_set_control(m_stream, SPA_PROP_channelVolumes, channels(), values, 0);
-    delete[] values;
+    delete [] values;
     pw_thread_loop_unlock(m_loop);
 }
 
@@ -430,7 +413,7 @@ void OutputPipeWire::onCoreEventDone(void *data, uint32_t id, int seq)
         spa_hook_remove(&o->m_registryListener);
         spa_hook_remove(&o->m_coreListener);
 
-        o->m_initDone = true;
+        o->m_inited = true;
         pw_thread_loop_signal(o->m_loop, false);
     }
 }
@@ -458,6 +441,7 @@ void OutputPipeWire::onRegistryEventGlobal(void *data, uint32_t id, uint32_t per
     o->m_coreInitSeq = pw_core_sync(o->m_core, PW_ID_CORE, o->m_coreInitSeq);
 }
 
+//volume control
 VolumePipeWire::VolumePipeWire()
 {
     QSettings settings(Qmmp::configFile(), QSettings::IniFormat);
diff --git a/src/plugins/Output/pipewire/outputpipewire.h b/src/plugins/Output/pipewire/outputpipewire.h
index a53612005..047fea894 100644
--- a/src/plugins/Output/pipewire/outputpipewire.h
+++ b/src/plugins/Output/pipewire/outputpipewire.h
@@ -25,7 +25,6 @@
 #include <QHash>
 #include <pipewire/pipewire.h>
 #include <spa/param/audio/format-utils.h>
-#include <atomic>
 #include <qmmp/output.h>
 #include <qmmp/volume.h>
 
@@ -48,6 +47,7 @@ public:
     void reset() override;
     void suspend() override;
     void resume() override;
+    // helper functions
     void setMuted(bool mute);
     void setVolume(const VolumeSettings &v);
 
@@ -62,12 +62,12 @@ private:
     static void onDrained(void *data);
     static void onCoreEventDone(void *data, uint32_t id, int seq);
     static void onRegistryEventGlobal(void *data, uint32_t id,
-                                      uint32_t permissions, const char *type, uint32_t version,
-                                      const struct spa_dict *props);
-
-    static void onStateChanged (void *data, enum pw_stream_state old,
-                           enum pw_stream_state state, const char *error);
+                                      uint32_t permissions, const char *type,
+                                      uint32_t version, const struct spa_dict *props);
+    static void onStateChanged(void *data, enum pw_stream_state old,
+                               enum pw_stream_state state, const char *error);
 
+    // PipeWire objects
     struct pw_thread_loop *m_loop = nullptr;
     struct pw_stream *m_stream  = nullptr;
     pw_context *m_context = nullptr;
@@ -77,28 +77,17 @@ private:
     spa_hook m_streamListener = {};
     spa_hook m_registryListener = {};
 
-    uint32_t m_stride = 0;
-    uint32_t m_nFrames = 0;
-    uint32_t m_bufferSize = 0;
-    uint32_t m_writeBufferPos = 0;
-
     int m_coreInitSeq = 0;
 
-    std::atomic_bool m_hasSinks {false};
-    std::atomic_bool m_initDone {false};
-    std::atomic_bool m_bufferHasData {false};
-    std::atomic_bool m_processed {false};
-    std::atomic_bool m_paused {false};
-    std::atomic_bool m_silence {false};
-    std::atomic_bool m_streamPaused {false};
-    std::atomic_bool m_ignoreStateChange {false};
-    std::atomic_bool m_err {false};
-
+    bool m_inited = false;
+    bool m_hasSinks = false;
+    bool m_ignoreStateChange = false;
 
-    //uint8_t m_stream_buffer[1024];
-    //unsigned char m_buffer[4096];
     unsigned char *m_buffer = nullptr;
-    uint32_t m_buffer_at = 0;
+    quint32 m_buffer_at = 0;
+    quint32 m_bufferSize = 0;
+    quint32 m_frames = 0;
+    quint32 m_stride = 0;
     QHash <Qmmp::ChannelPosition, spa_audio_channel> m_pw_channels;
 };
 
@@ -108,14 +97,11 @@ public:
     VolumePipeWire();
     ~VolumePipeWire();
 
-    //void updateVolume(const pa_cvolume &v, bool muted);
     void setVolume(const VolumeSettings &vol) override;
     VolumeSettings volume() const override;
     bool isMuted() const override;
     void setMuted(bool mute) override;
     VolumeFlags flags() const override;
-    //static VolumeSettings cvolumeToVolumeSettings(const pa_cvolume &v);
-    //static pa_cvolume volumeSettingsToCvolume(const VolumeSettings &v, int channels);
 
 private:
     VolumeSettings m_volume;