[MPlayer-dev-eng] [PATCH] af_scale* command-line only

[Scott Lamb]

Dominik 'Rathann' Mierzejewski wrote:
> On Thursday, 06 September 2007 at 23:53, Scott Lamb wrote:
> [...] 
>> * scalepitch= sounds great and almost does what I want:
>>
>> ** it seems to *increase* the pitch by the given factor, so when used 
>> with -speed, the numbers need to be reciprocals to maintain the original 
>> pitch. E.g., I am running it with the rather ugly command "mplayer -af 
>> scalepitch=.76923076923076923076 -speed 1.3 file.avi". It would be more 
>> convenient if I didn't have to use a calculator to generate the correct 
>> command.
> 
> Does -af scalepitch=10/13 -speed 13/10 file.avi work?

No, this code:

   case AF_CONTROL_COMMAND_LINE:{
     float f;
     sscanf((char*)arg,"%f", &f);
     return control(af, AF_CONTROL_SCALEPITCH_AMOUNT | AF_CONTROL_SET, &f);
   }

yields "10" in that case, which results in horrible shrieking.

Best regards,
Scott

-- 
Scott Lamb <http://www.slamb.org/>