[MPlayer-dev-eng] [PATCH] Proof of concept extrasurround plugin

Kis Gergely kisg at lme.linux.hu
Fri Apr 5 00:10:32 CEST 2002


On Thu, Apr 04, 2002 at 11:12:11PM +0200, Felix Buenemann wrote:
> On Thursday 04 April 2002 22:03, Kis Gergely wrote:
> > -> Included "quick workaround" for ao_plugin.c bug that prevented 6 channel
> >     output in many cases. This is not a true fix, it may cause problems for
> >     you.
> could you give a more detailed description about this?
In ao_plugin.c in the play() method in some cases when using 6 channel
output the ao_plugin_data.len is set to size(ao_plugin_data.data)-1
Because in case of s16_le format the sample size is always a dividable by 2
I added this quick and dirty and uncomplete and ugly code to ao_plugin.c
play()

if (ret_len % 2 !=0) {
	ret_len += 1;
} 

And it worked. But for another afmts it will probably not work as expected.

The real problem is probably in get_space() but my knowledge of the ao
architecture is very limited and I think somebody from the developer team
could fix it more accurately. 


Bye,
kisg



More information about the MPlayer-dev-eng mailing list