[FFmpeg-devel] [PATCH v5] Add support for Audible AAX (and AAX+) files
Vesselin Bontchev
vesselin.bontchev at yandex.com
Sat Jul 18 13:04:44 CEST 2015
18.07.2015, 13:03, "Paul B Mahol" <onemda at gmail.com>:
> Dana 18. 7. 2015. 11:46 osoba "Vesselin Bontchev" <
> vesselin.bontchev at yandex.com> napisala je:
>> 18.07.2015, 01:45, "Michael Niedermayer" <michael at niedermayer.cc>:
>> > On Thu, Jul 16, 2015 at 03:57:26AM +0300, Vesselin Bontchev wrote:
>> >> + // extracted from libAAX_SDK.so and AAXSDKWin.dll files!
>> >
>> >> + uint8_t fixed_key[] = { 0x77, 0x21, 0x4d, 0x4b, 0x19, 0x6a, 0x87,
>> >
>> > I think this should not be hardcoded but provided by the user via
>> > an AVOption.
>>
>> I fear that doing this will impact the usability negatively. Are there
>> strong reasons for suggesting this change?
> Using AVOption will set default key to what is already now hardcoded.
Thanks for explaining this! I have now uploaded a new revision (v6) of the patch, which uses an AVOption.
Vesselin
More information about the ffmpeg-devel
mailing list