[pulseaudio-discuss] [PATCH v2 0/4] Add "volume_is_absolute" in stream restore

[Marc-André Lureau]

Hello again,

The following patches introduces a new boolean flag to indicate
if the rule entry volume is absolute.

Client-server now send their extensions list they support. There is no
need to call ext_test() with the given solution (it is sent with
SET_CLIENT_NAME as suggested by Lennart, kudos:).

It does not break API/ABI, but introduces new functions and structure
as explained in the patches pa_ext_stream_restore2*.

I have run test with valgrind, client/server, and updated pavucontrol
to use the "stream-restore-2" extension, it works here (!). But the
changes where all over the code, and we were discussing several issues
with different solutions on IRC. Maybe the code would not have introduced
new strlist functions, for instance, if I knew exactly what it would look
like in the end. Hope you like it!

Let me conclude with: all that for a fucking bool! :-P

good evening,

Marc-André Lureau (4):
  stream-restore: extend ext-stream-restore
  stream-restore: add volume_is_absolute bool in server side
  protocol-native: add "ext_supported" to install_ext()
  native-protocol: send supported extensions in client & server

 PROTOCOL                            |   23 +++++++
 configure.ac                        |    2 +-
 src/map-file                        |    2 +
 src/modules/module-stream-restore.c |   32 +++++++---
 src/pulse/context.c                 |   49 ++++++++++++++-
 src/pulse/context.h                 |    3 +
 src/pulse/ext-stream-restore.c      |  117 +++++++++++++++++++++++++++++------
 src/pulse/ext-stream-restore.h      |   41 ++++++++++++-
 src/pulse/internal.h                |    2 +
 src/pulse/xmalloc.h                 |   13 ++++
 src/pulsecore/core-util.h           |   16 +++++
 src/pulsecore/native-common.h       |    1 +
 src/pulsecore/protocol-native.c     |  113 ++++++++++++++++++++++++++++++----
 src/pulsecore/protocol-native.h     |    5 +-
 src/pulsecore/strlist.c             |   37 +++++++++++
 src/pulsecore/strlist.h             |    9 +++
 src/tests/strlist-test.c            |    9 +++
 17 files changed, 428 insertions(+), 46 deletions(-)