On 11/6/2017 8:43 PM, Michael Niedermayer wrote:
> On Sun, Nov 05, 2017 at 03:06:56PM -0300, James Almer wrote:
>> It has no reason to be in a public header, even if defined as private.
>>
>> Signed-off-by: James Almer <jamr...@gmail.com>
>> ---
>> To be honest, none of the fields below the "not part of the API" notice
>> have any reason to be in the public struct, but this one is particularly
>> pointless and ugly to have there as FFFrac is defined in internal.h
>>
>>  libavformat/avformat.h |  2 --
>>  libavformat/internal.h |  2 ++
>>  libavformat/mux.c      | 18 +++++++++---------
>>  libavformat/utils.c    |  6 +++---
>>  4 files changed, 14 insertions(+), 14 deletions(-)
> 
> probably ok
> 
> thx

Pushed, thanks.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to