[pulseaudio-discuss] [PATCH 1/3] sink_input: new volume_factor system

Flávio Ceolin flavio.ceolin at profusion.mobi
Mon Nov 26 13:34:22 PST 2012


Hi Paul,

On Mon, Nov 26, 2012 at 7:10 PM, Paul Menzel
<paulepanter at users.sourceforge.net> wrote:
> Dear Flavio,
>
>
> thank you for your patches. As you are asked to resend anyway, here are
> some nitpicks.
>
>
> Am Mittwoch, den 14.11.2012, 15:19 -0200 schrieb Flavio Ceolin:
>> This patch makes possible to set more than one volume factor.  The
>
> s,makes,makes it,
>
> Also starting a commit message with »This patch« is redundant most of
> the time.
>
>> real value of the volume_factor will be the multiplication of these
>> values.
>
> Something like »Implement setting of more than one volume factor. …« or
> so could also be used.
>
>> ---
>>  src/pulsecore/sink-input.c | 172 ++++++++++++++++++++++++++++++++++++++-------
>>  src/pulsecore/sink-input.h |  19 +++--
>>  2 files changed, 158 insertions(+), 33 deletions(-)
>
> […]
>
>
> Thanks,
>
> Paul

Thank you for reviewing the patch, i going to fix these mistakes.

Best Regards,
Flavio Ceolin


More information about the pulseaudio-discuss mailing list