Re: [libav-devel] [PATCH 023/200] avcodecpar: switch to the new channel layout API

2017-06-07 Thread Luca Barbato 
On 6/7/17 8:04 PM, Vittorio Giovara wrote:
> On Wed, Jun 7, 2017 at 2:02 PM, Luca Barbato  wrote:
>> On 6/7/17 5:46 PM, Vittorio Giovara wrote:
>>> +return av_channel_layout_copy(>ch_layout, >ch_layout);
>>
>> Usual question regarding copy freeing or not the map before copying.
> 
> I believe we settled for clearing the map before copying.
> 

then it is ok.
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 023/200] avcodecpar: switch to the new channel layout API

2017-06-07 Thread Vittorio Giovara 
On Wed, Jun 7, 2017 at 2:02 PM, Luca Barbato  wrote:
> On 6/7/17 5:46 PM, Vittorio Giovara wrote:
>> +return av_channel_layout_copy(>ch_layout, >ch_layout);
>
> Usual question regarding copy freeing or not the map before copying.

I believe we settled for clearing the map before copying.
-- 
Vittorio
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 023/200] avcodecpar: switch to the new channel layout API

2017-06-07 Thread Luca Barbato 
On 6/7/17 5:46 PM, Vittorio Giovara wrote:
> +return av_channel_layout_copy(>ch_layout, >ch_layout);

Usual question regarding copy freeing or not the map before copying.
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel