Hi Ricardo and thanks for the patch,

Could you give some context - is this just a refactoring patch?

Because there is launchpad bug [1], and the suspicious text "Temporarily disabling fallback table as that can get pulse in a crash loop without the fix available in pulseaudio 1:4.0-0ubuntu21" but there is no 1:4.0-0ubuntu21 version of PulseAudio anywhere I can find...

On 2014-10-02 01:07, Ricardo Salveti de Araujo wrote:
From: Ricardo Salveti de Araujo <[email protected]>

Signed-off-by: Ricardo Salveti de Araujo <[email protected]>
---
  src/modules/module-stream-restore.c | 9 +--------
  1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/modules/module-stream-restore.c 
b/src/modules/module-stream-restore.c
index 38d6aac..06ab543 100644
--- a/src/modules/module-stream-restore.c
+++ b/src/modules/module-stream-restore.c
@@ -1831,7 +1831,6 @@ static int fill_db(struct userdata *u, const char 
*filename) {
          *d = 0;
          if (pa_atod(v, &db) >= 0) {
              if (db <= 0.0) {
-                pa_datum key, data;
                  struct entry e;

                  pa_zero(e);
@@ -1840,13 +1839,7 @@ static int fill_db(struct userdata *u, const char 
*filename) {
                  pa_cvolume_set(&e.volume, 1, pa_sw_volume_from_dB(db));
                  pa_channel_map_init_mono(&e.channel_map);

-                key.data = (void *) ln;
-                key.size = strlen(ln);
-
-                data.data = (void *) &e;
-                data.size = sizeof(e);
-
-                if (pa_database_set(u->database, &key, &data, false) == 0)
+                if (entry_write(u, ln, &e, false))
                      pa_log_debug("Setting %s to %0.2f dB.", ln, db);
              } else
                  pa_log_warn("[%s:%u] Positive dB values are not allowed, not 
setting entry %s.", fn, n, ln);


--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic

[1] https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1366819
_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss

Reply via email to