Re: [FFmpeg-devel] [PATCH] avcodec: remove unused avpriv_ac3_parse_header

2015-10-20 Thread Andreas Cadhalpun
On 14.10.2015 02:07, Michael Niedermayer wrote: > On Wed, Oct 14, 2015 at 01:50:13AM +0200, Andreas Cadhalpun wrote: >> It was replaced by avpriv_ac3_parse_header2. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/ac3_parser.c | 9 - >> libavcodec/ac3_parser.h | 2 -- >> 2 file

Re: [FFmpeg-devel] [PATCH] avcodec: remove unused avpriv_ac3_parse_header

2015-10-20 Thread Andreas Cadhalpun
On 14.10.2015 01:53, James Almer wrote: > On 10/13/2015 8:50 PM, Andreas Cadhalpun wrote: >> It was replaced by avpriv_ac3_parse_header2. > > You could rename it to avpriv_ac3_parse_header() while at it. > The 2 suffix becomes silly with this. I'll send a separate patch for that. Best regards, A

Re: [FFmpeg-devel] [PATCH] avcodec: remove unused avpriv_ac3_parse_header

2015-10-13 Thread Michael Niedermayer
On Wed, Oct 14, 2015 at 01:50:13AM +0200, Andreas Cadhalpun wrote: > It was replaced by avpriv_ac3_parse_header2. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/ac3_parser.c | 9 - > libavcodec/ac3_parser.h | 2 -- > 2 files changed, 11 deletions(-) should be ok [...] -- Mich

Re: [FFmpeg-devel] [PATCH] avcodec: remove unused avpriv_ac3_parse_header

2015-10-13 Thread James Almer
On 10/13/2015 8:50 PM, Andreas Cadhalpun wrote: > It was replaced by avpriv_ac3_parse_header2. You could rename it to avpriv_ac3_parse_header() while at it. The 2 suffix becomes silly with this. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http:

[FFmpeg-devel] [PATCH] avcodec: remove unused avpriv_ac3_parse_header

2015-10-13 Thread Andreas Cadhalpun
It was replaced by avpriv_ac3_parse_header2. Signed-off-by: Andreas Cadhalpun --- libavcodec/ac3_parser.c | 9 - libavcodec/ac3_parser.h | 2 -- 2 files changed, 11 deletions(-) diff --git a/libavcodec/ac3_parser.c b/libavcodec/ac3_parser.c index 5ab5627..0dd98b9 100644 --- a/libavcodec