[MPlayer-cvslog] CVS: main/libao2 ao_pcm.c,1.32,1.33
Corey Hickey
bugfood-ml at fatooh.org
Sun May 14 05:22:46 CEST 2006
Corey Hickey wrote:
> Diego Biurrun CVS wrote:
>> CVS change done by Diego Biurrun CVS
>>
>> Update of /cvsroot/mplayer/main/libao2
>> In directory mail:/var2/tmp/cvs-serv17304/libao2
>>
>> Modified Files:
>> ao_pcm.c
>> Log Message:
>> Move setting the output filename after the suboption parsing, otherwise it
>> cannot take the suboptions that were set into account.
>>
>>
>> Index: ao_pcm.c
>> ===================================================================
>> RCS file: /cvsroot/mplayer/main/libao2/ao_pcm.c,v
>> retrieving revision 1.32
>> retrieving revision 1.33
>> diff -u -r1.32 -r1.33
>> --- ao_pcm.c 14 Apr 2006 13:18:33 -0000 1.32
>> +++ ao_pcm.c 13 May 2006 19:04:39 -0000 1.33
>> @@ -89,12 +89,14 @@
>> };
>> // set defaults
>> ao_pcm_waveheader = 1;
>> - ao_outputfilename =
>> - strdup((ao_pcm_waveheader)?"audiodump.wav":"audiodump.pcm");
>> +
>> if (subopt_parse(ao_subdevice, subopts) != 0) {
>> return 0;
>> }
>>
>> + ao_outputfilename =
>> + strdup((ao_pcm_waveheader)?"audiodump.wav":"audiodump.pcm");
>> +
>> /* bits is only equal to format if (format == 8) or (format == 16);
>> this means that the following "if" is a kludge and should
>> really be a switch to be correct in all cases */
>
> I haven't verified for absolutely 100% sure, but it looks like this is
> breaking usage of the following:
>
> $ mplayer file.avi -ao pcm:file=audio.wav
>
> Instead of writing to audio.wav, mplayer makes audiodump.wav, ignore the
> file= suboption.
>
> -Corey
Patch attached. Please apply it or some better fix before pre8. :)
-Corey
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fix-ao-pcm-file.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20060513/22764a88/attachment.asc>
More information about the MPlayer-cvslog
mailing list