About volume ramping

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 30.12.09 16:20, huan zheng (huan.zheng.zbt at gmail.com) wrote:

> Hi,
Heya!

> I found that volume ramping is inside PA now.
> there's a minor fix here, it is a bug i found after the patch is submitted:
> 
> line 1781 of sink-input.c :
> if ((i->thread_info.ramp_info.envelope_dying - nbytes) <= 0) {
> need to be changed to
> if ((i->thread_info.ramp_info.envelope_dying - (ssize_t) nbytes) <= 0) {
> 
> otherwise this argument will never be negative since nbytes is of type
> size_t which is unsigned.
> Please change it when you have time, sorry if bring any inconvenience. :)

Thanks for the hint.

I have fixed that now. Note that I merged that but have not really
tested yet and my plan is to extend this to proper multichannel
ramping before the actual release.

(BTW: proper patches greatly preferred)

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net
http://0pointer.net/lennart/           GnuPG 0x1A015CC4



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux