Commit 2fcba50ec99e67c8efc65eb409525c346a2171d7
2021-02-27T12:08:15
audio: pipewire: Code and comment cleanups Replace "magic numbers" with #defines, explain the requirements when using the userdata pointer in the node_object struct and a few other minor code and comment cleanups.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64
diff --git a/src/audio/pipewire/SDL_pipewire.c b/src/audio/pipewire/SDL_pipewire.c
index d561327..68dac92 100644
--- a/src/audio/pipewire/SDL_pipewire.c
+++ b/src/audio/pipewire/SDL_pipewire.c
@@ -38,6 +38,9 @@
#define PW_MIN_SAMPLES 32 /* About 0.67ms at 48kHz */
#define PW_MAX_SAMPLES 8192 /* About 170.6ms at 48kHz */
+#define PW_POD_BUFFER_LENGTH 1024
+#define PW_THREAD_NAME_BUFFER_LENGTH 128
+
#define PW_ID_TO_HANDLE(x) (void *)((uintptr_t)x)
#define PW_HANDLE_TO_ID(x) (uint32_t)((uintptr_t)x)
@@ -195,7 +198,15 @@ struct node_object
Uint32 id;
int seq;
- void * userdata;
+
+ /*
+ * NOTE: If used, this is *must* be allocated with SDL_malloc() or similar
+ * as SDL_free() will be called on it when the node_object is destroyed.
+ *
+ * If ownership of the referenced memory is transferred, this must be set
+ * to NULL or the memory will be freed when the node_object is destroyed.
+ */
+ void *userdata;
struct pw_proxy *proxy;
struct spa_hook node_listener;
@@ -986,15 +997,15 @@ PIPEWIRE_OpenDevice(_THIS, void *handle, const char *devname, int iscapture)
{
/*
* NOTE: The PW_STREAM_FLAG_RT_PROCESS flag can be set to call the stream
- * processing callback from the realtime thread, however it comes with some
+ * processing callback from the realtime thread. However, it comes with some
* caveats: no file IO, allocations, locking or other blocking operations
* must occur in the mixer callback. As this cannot be guaranteed when the
* callback is in the calling application, this flag is omitted.
*/
static const enum pw_stream_flags STREAM_FLAGS = PW_STREAM_FLAG_AUTOCONNECT | PW_STREAM_FLAG_MAP_BUFFERS;
- char thread_name[128];
- Uint8 pod_buffer[1024];
+ char thread_name[PW_THREAD_NAME_BUFFER_LENGTH];
+ Uint8 pod_buffer[PW_POD_BUFFER_LENGTH];
struct spa_pod_builder b = SPA_POD_BUILDER_INIT(pod_buffer, sizeof(pod_buffer));
struct spa_audio_info_raw spa_info = { 0 };
const struct spa_pod * params = NULL;
@@ -1032,12 +1043,10 @@ PIPEWIRE_OpenDevice(_THIS, void *handle, const char *devname, int iscapture)
min_period = PW_MIN_SAMPLES * SPA_MAX(this->spec.freq / 48000, 1);
adjusted_samples = SPA_CLAMP(this->spec.samples, min_period, PW_MAX_SAMPLES);
- if ((priv = SDL_calloc(1, sizeof(struct SDL_PrivateAudioData))) == NULL) {
+ if ((this->hidden = priv = SDL_calloc(1, sizeof(struct SDL_PrivateAudioData))) == NULL) {
return SDL_OutOfMemory();
}
- this->hidden = priv;
-
/* Size of a single audio frame in bytes */
priv->stride = (SDL_AUDIO_BITSIZE(this->spec.format) >> 3) * this->spec.channels;