[pulseaudio-discuss] [PATCH 1/3] conf-parser: Pass parser state in a struct instead of function parameters.

Tanu Kaskinen tanuk at iki.fi
Mon Dec 5 10:20:04 PST 2011


On Sat, 2011-12-03 at 12:58 +0100, Maarten Bosmans wrote:
> 2011/12/3 Tanu Kaskinen <tanuk at iki.fi>:
> > I don't like long function parameter lists, and I plan to
> > add some more state data to the parser which would make the
> > parameter lists even longer without this refactoring.
> 
> Why not also change the pa_config_parser_cb_t prototype to accept a
> parser_state struct?

Good idea. I'll add that to the todo list.

-- 
Tanu



More information about the pulseaudio-discuss mailing list