[pulseaudio-discuss] [PATCH 0/4] Third try at making port properties configurable

Tanu Kaskinen tanu.kaskinen at digia.com
Tue Apr 10 07:02:31 PDT 2012


Back in December I sent a patch series that implemented
configurable port properties:
http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/12070

David Henningsson pointed out that having a separate
"Property List" section would be nicer syntax than having
just one option in the "General" section containing all
properties. I implemented that then:
http://thread.gmane.org/gmane.comp.audio.pulseaudio.general/12075

This time David suggested that "Properties" would be
a better section name than "Property List". Also, Maarten
Bosmans suggested further refactoring in the configuration
parsing code: the parse callbacks could also take the parser
state struct as a parameter, instead of passing all
the state information in separate parameters. This third
patch set implements those suggestions.

Tanu Kaskinen (4):
  conf-parser: Pass parser state in a struct instead of function
    parameters.
  conf-parser: Pass parser state in a struct also for parse callbacks.
  conf-parser: Add support for parsing property lists.
  alsa-mixer: Add support for defining port property lists in the path
    configuration files.

 src/daemon/daemon-conf.c                           |  222 ++++----
 src/modules/alsa/alsa-mixer.c                      |  594 ++++++++------------
 src/modules/alsa/alsa-mixer.h                      |    1 +
 .../alsa/mixer/paths/analog-output.conf.common     |    4 +
 src/modules/module-augment-properties.c            |   62 +--
 src/pulse/client-conf.c                            |    2 +-
 src/pulsecore/conf-parser.c                        |  221 ++++----
 src/pulsecore/conf-parser.h                        |   48 ++-
 8 files changed, 532 insertions(+), 622 deletions(-)

-- 
1.7.8



More information about the pulseaudio-discuss mailing list