[pulseaudio-discuss] [PATCH] device-port: Change the latency offset type to a signed int.
poljar (Damir Jelić)
poljarinho at gmail.com
Wed Jun 27 08:38:42 PDT 2012
From: "poljar (Damir Jelić)" <poljarinho at gmail.com>
The latency offset type should be signed (int64_t) so we can also add
a negative latency offset.
This also includes changing the type of the sink/source
offsets and updating pacmd so it handles negative numbers.
---
src/pulsecore/cli-command.c | 2 +-
src/pulsecore/device-port.c | 2 +-
src/pulsecore/device-port.h | 4 ++--
src/pulsecore/sink.c | 6 +++---
src/pulsecore/sink.h | 6 +++---
src/pulsecore/source.c | 6 +++---
src/pulsecore/source.h | 6 +++---
7 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/src/pulsecore/cli-command.c b/src/pulsecore/cli-command.c
index 33f4d49..a772e1e 100644
--- a/src/pulsecore/cli-command.c
+++ b/src/pulsecore/cli-command.c
@@ -1766,7 +1766,7 @@ static int pa_cli_command_port_offset(pa_core *c, pa_tokenizer *t, pa_strbuf *bu
return -1;
}
- pa_device_port_set_latency_offset(port, (pa_usec_t) offset);
+ pa_device_port_set_latency_offset(port, offset);
return 0;
}
diff --git a/src/pulsecore/device-port.c b/src/pulsecore/device-port.c
index 46e37e2..28879f7 100644
--- a/src/pulsecore/device-port.c
+++ b/src/pulsecore/device-port.c
@@ -114,7 +114,7 @@ void pa_device_port_hashmap_free(pa_hashmap *h) {
pa_hashmap_free(h, NULL, NULL);
}
-void pa_device_port_set_latency_offset(pa_device_port *p, pa_usec_t offset) {
+void pa_device_port_set_latency_offset(pa_device_port *p, int64_t offset) {
uint32_t state;
pa_assert(p);
diff --git a/src/pulsecore/device-port.h b/src/pulsecore/device-port.h
index 5880195..a5c6420 100644
--- a/src/pulsecore/device-port.h
+++ b/src/pulsecore/device-port.h
@@ -51,7 +51,7 @@ struct pa_device_port {
pa_hashmap *profiles; /* Does not own the profiles */
pa_bool_t is_input:1;
pa_bool_t is_output:1;
- pa_usec_t latency_offset;
+ int64_t latency_offset;
/* .. followed by some implementation specific data */
};
@@ -68,6 +68,6 @@ void pa_device_port_hashmap_free(pa_hashmap *h);
/* The port's available status has changed */
void pa_device_port_set_available(pa_device_port *p, pa_port_available_t available);
-void pa_device_port_set_latency_offset(pa_device_port *p, pa_usec_t offset);
+void pa_device_port_set_latency_offset(pa_device_port *p, int64_t offset);
#endif
diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
index 545a8e1..44d3c7f 100644
--- a/src/pulsecore/sink.c
+++ b/src/pulsecore/sink.c
@@ -2823,7 +2823,7 @@ int pa_sink_process_msg(pa_msgobject *o, int code, void *userdata, int64_t offse
return 0;
case PA_SINK_MESSAGE_SET_LATENCY_OFFSET:
- s->thread_info.latency_offset = (pa_usec_t) offset;
+ s->thread_info.latency_offset = offset;
return 0;
case PA_SINK_MESSAGE_GET_LATENCY:
@@ -3238,13 +3238,13 @@ void pa_sink_set_fixed_latency_within_thread(pa_sink *s, pa_usec_t latency) {
}
/* Called from main context */
-void pa_sink_set_latency_offset(pa_sink *s, pa_usec_t offset) {
+void pa_sink_set_latency_offset(pa_sink *s, int64_t offset) {
pa_sink_assert_ref(s);
s->latency_offset = offset;
if (PA_SINK_IS_LINKED(s->state))
- pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SINK_MESSAGE_SET_LATENCY_OFFSET, NULL, (int64_t) offset, NULL) == 0);
+ pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SINK_MESSAGE_SET_LATENCY_OFFSET, NULL, offset, NULL) == 0);
else
s->thread_info.fixed_latency = offset;
}
diff --git a/src/pulsecore/sink.h b/src/pulsecore/sink.h
index b138a1d..2c52348 100644
--- a/src/pulsecore/sink.h
+++ b/src/pulsecore/sink.h
@@ -114,7 +114,7 @@ struct pa_sink {
pa_atomic_t mixer_dirty;
/* The latency offset is inherited from the currently active port */
- pa_usec_t latency_offset;
+ int64_t latency_offset;
unsigned priority;
@@ -272,7 +272,7 @@ struct pa_sink {
pa_usec_t fixed_latency; /* for sinks with PA_SINK_DYNAMIC_LATENCY this is 0 */
/* This latency offset is a direct copy from s->latency_offset */
- pa_usec_t latency_offset;
+ int64_t latency_offset;
/* Delayed volume change events are queued here. The events
* are stored in expiration order. The one expiring next is in
@@ -410,7 +410,7 @@ unsigned pa_device_init_priority(pa_proplist *p);
/**** May be called by everyone, from main context */
pa_bool_t pa_sink_update_rate(pa_sink *s, uint32_t rate, pa_bool_t passthrough);
-void pa_sink_set_latency_offset(pa_sink *s, pa_usec_t offset);
+void pa_sink_set_latency_offset(pa_sink *s, int64_t offset);
/* The returned value is supposed to be in the time domain of the sound card! */
pa_usec_t pa_sink_get_latency(pa_sink *s);
diff --git a/src/pulsecore/source.c b/src/pulsecore/source.c
index 63a75ef..0b8a60a 100644
--- a/src/pulsecore/source.c
+++ b/src/pulsecore/source.c
@@ -2179,7 +2179,7 @@ int pa_source_process_msg(pa_msgobject *object, int code, void *userdata, int64_
return 0;
case PA_SOURCE_MESSAGE_SET_LATENCY_OFFSET:
- s->thread_info.latency_offset = (pa_usec_t) offset;
+ s->thread_info.latency_offset = offset;
return 0;
case PA_SOURCE_MESSAGE_MAX:
@@ -2522,13 +2522,13 @@ void pa_source_set_fixed_latency_within_thread(pa_source *s, pa_usec_t latency)
}
/* Called from main thread */
-void pa_source_set_latency_offset(pa_source *s, pa_usec_t offset) {
+void pa_source_set_latency_offset(pa_source *s, int64_t offset) {
pa_source_assert_ref(s);
s->latency_offset = offset;
if (PA_SOURCE_IS_LINKED(s->state))
- pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_LATENCY_OFFSET, NULL, (int64_t) offset, NULL) == 0);
+ pa_assert_se(pa_asyncmsgq_send(s->asyncmsgq, PA_MSGOBJECT(s), PA_SOURCE_MESSAGE_SET_LATENCY_OFFSET, NULL, offset, NULL) == 0);
else
s->thread_info.fixed_latency = offset;
}
diff --git a/src/pulsecore/source.h b/src/pulsecore/source.h
index da024e6..dd1e09b 100644
--- a/src/pulsecore/source.h
+++ b/src/pulsecore/source.h
@@ -114,7 +114,7 @@ struct pa_source {
pa_atomic_t mixer_dirty;
/* The latency offset is inherited from the currently active port */
- pa_usec_t latency_offset;
+ int64_t latency_offset;
unsigned priority;
@@ -213,7 +213,7 @@ struct pa_source {
pa_usec_t fixed_latency; /* for sources with PA_SOURCE_DYNAMIC_LATENCY this is 0 */
/* This latency offset is a direct copy from s->latency_offset */
- pa_usec_t latency_offset;
+ int64_t latency_offset;
/* Delayed volume change events are queued here. The events
* are stored in expiration order. The one expiring next is in
@@ -342,7 +342,7 @@ void pa_source_update_flags(pa_source *s, pa_source_flags_t mask, pa_source_flag
/*** May be called by everyone, from main context */
-void pa_source_set_latency_offset(pa_source *s, pa_usec_t offset);
+void pa_source_set_latency_offset(pa_source *s, int64_t offset);
/* The returned value is supposed to be in the time domain of the sound card! */
pa_usec_t pa_source_get_latency(pa_source *s);
--
1.7.11.1
More information about the pulseaudio-discuss
mailing list