On 02/10/15 14:13, Kostya Shishkov wrote:
> On Fri, Oct 02, 2015 at 02:07:22PM +0200, Vittorio Giovara wrote:
>> On Fri, Oct 2, 2015 at 1:48 PM, Alexandra Hájková
>> <[email protected]> wrote:
>>> place primary audio coding header data into DCAAudioHeader
>>> structure and channel related data to DCAChan structure
>>> ---
>>>  libavcodec/dca.h    |  70 ++++++++-----
>>>  libavcodec/dcadec.c | 290 
>>> ++++++++++++++++++++++++++++------------------------
>>>  2 files changed, 201 insertions(+), 159 deletions(-)
>>
>> Why is this needed? A little explanation in the commit log would be
>> nice (especially for big changes like this one).
> 
> "Because Kostya told so" would not make a good explanation :P

>From what I can see it is refactoring the core channels decoding. I
guess DCAChan will be used for the ext-channels as well.

DCAAudioHeader seems to be just a tidy up.

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to