[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