[pulseaudio-discuss] [PATCH v3 00/17] Format negotiation fixing
Arun Raghavan
arun.raghavan at collabora.co.uk
Thu Jan 9 21:22:01 PST 2014
On Wed, 2014-01-08 at 21:50 +0200, Tanu Kaskinen wrote:
> Link to the cover letter of the previous version (v2):
> http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/19232
>
> Changes in v3:
> - Dropped pa_stream_set_volume_channel_map(), as suggested by
> Arun. The function would only be useful in rare corner cases,
> so perhaps it's better to leave it out to avoid unnecessary
> clutter in the API. If someone requests this functionality, we
> can add it later.
> - Removed empty lines before checking r in code like this:
> r = foo();
>
> if (r < 0)
> do_something();
> - Initialize format to NULL in the beginning of
> pa_format_info_from_sample_spec2().
>
> Tanu Kaskinen (17):
> Move pa_format_info_to_sample_spec_fake() to core-format
> core-format: Add pa_format_info_get_sample_format()
> core-format: Add pa_format_info_get_rate()
> core-format: Add pa_format_info_get_channels()
> core-format: Add pa_format_info_get_channel_map()
> format: Simplify pa_format_info_to_sample_spec()
> core-format: Add pa_format_info_to_sample_spec2()
> core-format: Add pa_format_info_from_sample_spec2()
> sink-input, source-output: Do routing related validity checks
> immediately after routing
> format, core-format: Constify some function parameters
> stream-util: Add pa_stream_get_volume_channel_map()
> stream: Remove a volume channel validity check
> sink-input, source-output: Interpret missing PCM parameters in format
> info as a request to decide those parameters at the server end
> def, format: Document how to leave PCM parameters to be decided by the
> server
> stream: Improve pa_stream_connect_playback() documentation
> stream: Mention pa_stream_new_extended() in the high-level stream
> creation documentation
> format: Add some error logging
>
> src/Makefile.am | 2 +
> src/pulse/def.h | 38 ++++++-
> src/pulse/format.c | 116 ++++++++------------
> src/pulse/format.h | 74 ++++++++++---
> src/pulse/stream.c | 1 -
> src/pulse/stream.h | 17 ++-
> src/pulsecore/core-format.c | 243 ++++++++++++++++++++++++++++++++++++++++++
> src/pulsecore/core-format.h | 81 ++++++++++++++
> src/pulsecore/sink-input.c | 103 ++++++++----------
> src/pulsecore/source-output.c | 98 +++++++----------
> src/pulsecore/stream-util.c | 86 +++++++++++++++
> src/pulsecore/stream-util.h | 50 +++++++++
> 12 files changed, 695 insertions(+), 214 deletions(-)
> create mode 100644 src/pulsecore/core-format.c
> create mode 100644 src/pulsecore/core-format.h
> create mode 100644 src/pulsecore/stream-util.c
> create mode 100644 src/pulsecore/stream-util.h
Did a quick review - these look fine to me.
Thanks,
Arun
More information about the pulseaudio-discuss
mailing list