[FFmpeg-devel] [PATCH] lavc/rawdec: Remove superfluous AV_PIX_FMT_MONOWHITE code

2016-01-29 Thread Mats Peterson
Now when both AVI and QuickTime use pal8 for 1 bpp video, there's no need to keep the monow stuff. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 7a6627e283392703a1f90d680cc87761afa452e7 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sat, 30 Jan 2016 05:05:03 +0100 Subject: [

[FFmpeg-devel] [WEB PATCH] Add news entry on removal of libvo-aacenc and libaacplus

2016-01-29 Thread Timothy Gu
--- src/index | 41 + 1 file changed, 41 insertions(+) diff --git a/src/index b/src/index index 8680ef0..5c7afcd 100644 --- a/src/index +++ b/src/index @@ -37,6 +37,47 @@ News + January 30, 2016, Removing support for two external AAC encoders +

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libaacplus

2016-01-29 Thread Timothy Gu
On Thu, Jan 28, 2016 at 07:47:40PM -0800, Timothy Gu wrote: > > If I see no more comments, I will push this patch in one or two days. Pushed with the libvo-aacenc removal patch. I will add a website news entry soon. Timothy ___ ffmpeg-devel mailing li

Re: [FFmpeg-devel] FFmpeg 2.8.6 and others

2016-01-29 Thread Michael Niedermayer
On Sat, Jan 30, 2016 at 12:49:54AM +0100, Andreas Cadhalpun wrote: > On 27.01.2016 23:20, Michael Niedermayer wrote: > > ill make new point releases soon > > if you want something backported, backport it now > > > > also should i add the concat/subfile_enable switch and have them > > disabled by d

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

2016-01-29 Thread Timothy Gu
On Fri, Jan 29, 2016 at 11:43:53AM -0300, Claudio Freire wrote: > On Fri, Jan 29, 2016 at 1:08 AM, Timothy Gu wrote: > > > > Ping. > > Since it LGTM and there seems to be consensus, I'll push it when I get > a chance if nobody beats me to it. > > But ATM my hands are tied. I have lots of changes

[FFmpeg-devel] [PATCH 1/2] avformat: add protocol_whitelist

2016-01-29 Thread Michael Niedermayer
From: Michael Niedermayer TODO: Docs TODO: version bump Note to maintainers: update tools Note to maintainers: set a default whitelist for your protocol If that makes no sense then consider to set "none" and thus require the user to specify a white-list for sub-protocols to be opened Note, te

[FFmpeg-devel] [PATCH 2/2] Update demuxers and protocols for protocol whitelist support

2016-01-29 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavdevice/lavfi.c |7 ++- libavformat/async.c |2 +- libavformat/cache.c |3 ++- libavformat/concat.c |6 -- libavformat/crypto.c |5

Re: [FFmpeg-devel] [PATCH 3/3] Update demuxers and protocols for protocol whitelist support

2016-01-29 Thread Michael Niedermayer
On Sat, Jan 30, 2016 at 12:03:48AM +0100, Andreas Cadhalpun wrote: > On 28.01.2016 23:49, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > > libavdevice/lavfi.c |7 ++- > > libavformat/async.c |

Re: [FFmpeg-devel] [PATCH] avfilter: add nnedi filter

2016-01-29 Thread Andy Furniss
Paul B Mahol wrote: Hi, 3rd version attached! It seems that now bff/tff flags on input are correctly followed and output is good, but it can't be overridden by field=bf or field=tf both give correct output. For raw video though, they are honored and are still the wrong way round.

Re: [FFmpeg-devel] [PATCH v2 00/16] Replace native DCA decoder with libdcadec based one

2016-01-29 Thread Andreas Cadhalpun
On 25.01.2016 23:47, Hendrik Leppkes wrote: > The decoder in itself looks fine to me, short of the regression michael found. > If you could look at that, that would be great. > > I can squash and re-shuffle the commits appropriately for pushing and > make sure all intermediate steps still build, s

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-01-29 Thread Andreas Cadhalpun
On 28.01.2016 15:42, Michael Niedermayer wrote: > On Wed, Jan 27, 2016 at 11:52:28PM +0100, Andreas Cadhalpun wrote: >> Makefile |5 - >> configure |2 +- >> 2 files changed, 5 insertions(+), 2 deletions(-) >> 67bd05c13361082708ccc24ee5828c4c00a443a3 >> 0001-configure-fall-back-to-u

Re: [FFmpeg-devel] FFmpeg 2.8.6 and others

2016-01-29 Thread Andreas Cadhalpun
On 27.01.2016 23:20, Michael Niedermayer wrote: > ill make new point releases soon > if you want something backported, backport it now > > also should i add the concat/subfile_enable switch and have them > disabled by default or is it preferred to leave them as is? I'd rather have the protocol_wh

Re: [FFmpeg-devel] [PATCH 2/2] avformat: add protocol_whitelist

2016-01-29 Thread Andreas Cadhalpun
On 28.01.2016 20:16, Nicolas George wrote: > Well, since we are sure there are more information leak issues in this area > of code, we can still discuss a proper fix before the next one becomes > public. > > A few unsorted thoughts about it: > > - Restrictions can not be limited to just the proto

Re: [FFmpeg-devel] [PATCH 3/3] Update demuxers and protocols for protocol whitelist support

2016-01-29 Thread Andreas Cadhalpun
On 28.01.2016 23:49, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavdevice/lavfi.c |7 ++- > libavformat/async.c |2 +- > libavformat/cache.c |3 ++- > libavformat/concat.c

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/amrwbdec: use av_mod_uintp2

2016-01-29 Thread James Almer
On 1/29/2016 7:30 PM, Michael Niedermayer wrote: > On Wed, Jan 20, 2016 at 03:04:05PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/amrwbdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM > > thx All three pushed. ___

Re: [FFmpeg-devel] [PATCHv2] mpegtsenc: Do not fail ADTS AAC muxing if the first frame is not ADTS

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 10:41:14PM +0100, Michael Niedermayer wrote: > On Thu, Jan 14, 2016 at 07:37:21AM +0100, Hagen Schmidt wrote: > > On 15/12/15 08:00, Hagen Schmidt wrote: > > >Fixes ticket 279. > > >--- > > > libavformat/mpegtsenc.c | 2 -- > > > 1 file changed, 2 deletions(-) > > > > > >di

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add protocol_whitelist

2016-01-29 Thread Andreas Cadhalpun
On 29.01.2016 00:03, Michael Niedermayer wrote: > On Thu, Jan 28, 2016 at 11:58:37PM +0100, Michael Niedermayer wrote: >> ive added backwarding to AVFormatContext, this requires manual >> overiding for rtp at least though > > this can probably be avoided by adding rtp&udp to the http(s) default >

Re: [FFmpeg-devel] [PATCH 2/3] avformat: add protocol_whitelist

2016-01-29 Thread Andreas Cadhalpun
On 28.01.2016 23:49, Michael Niedermayer wrote: > From: Michael Niedermayer > > TODO: Docs > TODO: version bump > > Note to maintainers: update tools > > Note to maintainers: set a default whitelist for your protocol > If that makes no sense then consider to set "none" and thus require the user

Re: [FFmpeg-devel] [PATCH] avfilter: add nnedi filter

2016-01-29 Thread Paul B Mahol
Hi, 3rd version attached! From f7f9cd51e3c22651d4b6d3636c26e9e3b63f6db6 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 23 Jan 2016 17:15:53 +0100 Subject: [PATCH] avfilter: add nnedi filter Port of nnedi3 vapoursynth filter. Signed-off-by: Paul B Mahol --- configure|

Re: [FFmpeg-devel] [PATCH 4/5] avcodec/proresdec_lgpl: use av_mod_uintp2

2016-01-29 Thread Michael Niedermayer
On Wed, Jan 20, 2016 at 03:04:07PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/proresdec_lgpl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/proresdec_lgpl.c b/libavcodec/proresdec_lgpl.c > index 9e5674e..467a423 100644 > --- a

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/amrwbdec: use av_mod_uintp2

2016-01-29 Thread Michael Niedermayer
On Wed, Jan 20, 2016 at 03:04:05PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/amrwbdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algo

Re: [FFmpeg-devel] [PATCHv2] mpegtsenc: Do not fail ADTS AAC muxing if the first frame is not ADTS

2016-01-29 Thread Michael Niedermayer
On Thu, Jan 14, 2016 at 07:37:21AM +0100, Hagen Schmidt wrote: > On 15/12/15 08:00, Hagen Schmidt wrote: > >Fixes ticket 279. > >--- > > libavformat/mpegtsenc.c | 2 -- > > 1 file changed, 2 deletions(-) > > > >diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c > >index 19032f3..a0dca4

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: Allow files where the PreviousTagSize is not set according to the spec.

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 10:47:49AM -0800, Thierry Foucu wrote: > Some muxer use the FLV field PreviousTagSize to be the sum of tag > length. Without this change, the flv demuxer think the file is broken > and the re-sync will fail. > --- > libavformat/flvdec.c | 6 +- > 1 file changed, 5 inser

Re: [FFmpeg-devel] [PATCH] libavformat/network: use defined constant in poll

2016-01-29 Thread Michael Niedermayer
On Wed, Jan 27, 2016 at 11:08:37AM +0800, Zhao Zhili wrote: > From: Zhao Zhili > > --- > libavformat/network.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into demo

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: free url/avio options

2016-01-29 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 11:49:06PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavformat/avio.c|1 + > libavformat/aviobuf.c |1 + > 2 files changed, 2 insertions(+) applied [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] lavf/flvdec: Allow files where the PreviousTagSize is not set according to the spec.

2016-01-29 Thread Thierry Foucu
Some muxer use the FLV field PreviousTagSize to be the sum of tag length. Without this change, the flv demuxer think the file is broken and the re-sync will fail. --- libavformat/flvdec.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/flvdec.c b/libavformat/fl

Re: [FFmpeg-devel] [PATCH] libavutil/mastering_display_metadata.h: change fields to be rationals as this is how they are typically coded.

2016-01-29 Thread Michael Niedermayer
On Tue, Jan 26, 2016 at 04:33:56PM -0800, Neil Birkbeck wrote: > Patch with corrected "From:" is attached. > > On Tue, Jan 26, 2016 at 2:28 PM, Vittorio Gambaletta (VittGam) > wrote: > > On 26/01/2016 23:05:22 CET, Neil Birkbeck wrote: > >> > >> Some sort of squash fail. Apologies. > > > > > > No

Re: [FFmpeg-devel] [PATCH]lavc/v210dec: Accept odd width

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 04:45:45PM +, Carl Eugen Hoyos wrote: > Carl Eugen Hoyos ag.or.at> writes: > > > Michael Niedermayer niedermayer.cc> writes: > > > > > On Fri, Jan 29, 2016 at 11:20:18AM +0100, Carl Eugen Hoyos wrote: > > > > Attached patch fixes ticket #5195 for me. > > > > > > v2

[FFmpeg-devel] Need help with libswscale pal8 to monow issue

2016-01-29 Thread Mats Peterson
Conversion from pal8 to monow seems to be somewhat erratic. It produces slightly different output from converting from monow to monow. Pull the latest from the repository, get the files at the link below, and run the `mkraw' shell script. Comparing the newly produced file with the reference vsy

Re: [FFmpeg-devel] [PATCH 5/5] lavc/rawdec: Use AV_PIX_FMT_PAL8 for raw 1 bpp video in AVI

2016-01-29 Thread Mats Peterson
On 01/29/2016 05:00 PM, Michael Niedermayer wrote: patchset applied Fine, thanks. Now only the pal8 to monow conversion issue left to solve... Mats ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCHv2] Revert "do not write f2 if not interlaced"

2016-01-29 Thread Sebastian Dröge
From: Sebastian Dröge This reverts commit 8ed82d8174a666f80ab8834e3617cbe91ae740a9. SMPTE S377-1-2009c defines in F.4.1 that the Video Line Map should always be an array with two 32 bit integers as elements. This is repeated in G.2.12 with actual examples for progressive content, where the secon

Re: [FFmpeg-devel] [PATCH]lavc/v210dec: Accept odd width

2016-01-29 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Michael Niedermayer niedermayer.cc> writes: > > > On Fri, Jan 29, 2016 at 11:20:18AM +0100, Carl Eugen Hoyos wrote: > > > Attached patch fixes ticket #5195 for me. > > > > v210dec.c |6 +- > > > 1 file changed, 1 insertion(+), 5 deletions(-) > > >

Re: [FFmpeg-devel] Fwd: libavformat/segment : add option to increment timecode

2016-01-29 Thread Martin Vignali
2016-01-29 12:35 GMT+01:00 Stefano Sabatini : > On date Friday 2016-01-29 12:18:02 +0100, Martin Vignali encoded: > > Ping > > > > -- Forwarded message -- > > From: Martin Vignali > > Date: 2015-10-03 15:49 GMT+02:00 > > Subject: libavformat/segment : add option to increment timec

Re: [FFmpeg-devel] [PATCH 5/5] lavc/rawdec: Use AV_PIX_FMT_PAL8 for raw 1 bpp video in AVI

2016-01-29 Thread Michael Niedermayer
On Thu, Jan 28, 2016 at 06:19:28AM +0100, Mats Peterson wrote: > From > https://msdn.microsoft.com/en-us/library/windows/desktop/dd318229%28v=vs.85%29.aspx: > > "If biCompression equals BI_RGB and the bitmap uses 8 bpp or less, > the bitmap has a color table immediatelly following the > BITMAPINF

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libvo-aacenc support.

2016-01-29 Thread Claudio Freire
On Fri, Jan 29, 2016 at 1:08 AM, Timothy Gu wrote: > On Sun, Jan 24, 2016 at 04:33:31PM +, Kieran Kunhya wrote: >> The internal encoder is superior to libvo-aacenc. >> --- >> configure | 6 -- >> doc/encoders.texi | 25 -- >> doc/general.texi | 8 -- >

Re: [FFmpeg-devel] [PATCH] Revert "do not write f2 if not interlaced"

2016-01-29 Thread Sebastian Dröge
On Fr, 2016-01-29 at 13:31 +0100, Michael Niedermayer wrote: > > --- > >  libavformat/mxfenc.c | 8 > >  1 file changed, 4 insertions(+), 4 deletions(-) > > breaks fate > (you need to update the checksums if these chnages are intended) Ah sure, forgot to update those. Thanks! The checksum

Re: [FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-29 Thread Ari Koivula
On Fri, Jan 29, 2016 at 1:33 PM, Hendrik Leppkes wrote: > You could of course pack both field-pictures into the same AVPacket, > but that doesn't yield you proper timestamps for the second, and will > probably also screw up a bunch of other things later on. > Packing both pictures into the same

Re: [FFmpeg-devel] [PULL|TRIVIAL] libavdevice/decklink_common.h: fix broken build due to missing `/`

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 07:17:41AM -0500, Michael Ira Krufky wrote: > The build in the master branch is currently broken due to a missing > `/` in a comment introduced by commit > 44304ae3220f553d0b1458644e2a617ea1ad8d22 > > Please merge my branch or apply the 1-liner patch below > > The followin

Re: [FFmpeg-devel] [PATCH] Revert "do not write f2 if not interlaced"

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 12:39:03PM +0100, Sebastian Dröge wrote: > From: Sebastian Dröge > > This reverts commit 8ed82d8174a666f80ab8834e3617cbe91ae740a9. > > SMPTE S377-1-2009c defines in F.4.1 that the Video Line Map should > always be an array with two 32 bit integers as elements. This is > r

[FFmpeg-devel] [PULL|TRIVIAL] libavdevice/decklink_common.h: fix broken build due to missing `/`

2016-01-29 Thread Michael Ira Krufky
The build in the master branch is currently broken due to a missing `/` in a comment introduced by commit 44304ae3220f553d0b1458644e2a617ea1ad8d22 Please merge my branch or apply the 1-liner patch below The following changes since commit b91093a4111ffc947f3387325a377bb1694a87dd: ffmpeg: replac

Re: [FFmpeg-devel] [PATCH]lavc/v210dec: Accept odd width

2016-01-29 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > On Fri, Jan 29, 2016 at 11:20:18AM +0100, Carl Eugen Hoyos wrote: > > Attached patch fixes ticket #5195 for me. > > v210dec.c |6 +- > > 1 file changed, 1 insertion(+), 5 deletions(-) > > 9dcf44c7054d5443b80673b9717f22fa8c14c48b patchv210.d

[FFmpeg-devel] [PATCH] Revert "do not write f2 if not interlaced"

2016-01-29 Thread Sebastian Dröge
From: Sebastian Dröge This reverts commit 8ed82d8174a666f80ab8834e3617cbe91ae740a9. SMPTE S377-1-2009c defines in F.4.1 that the Video Line Map should always be an array with two 32 bit integers as elements. This is repeated in G.2.12 with actual examples for progressive content, where the secon

Re: [FFmpeg-devel] Fwd: libavformat/segment : add option to increment timecode

2016-01-29 Thread Stefano Sabatini
On date Friday 2016-01-29 12:18:02 +0100, Martin Vignali encoded: > Ping > > -- Forwarded message -- > From: Martin Vignali > Date: 2015-10-03 15:49 GMT+02:00 > Subject: libavformat/segment : add option to increment timecode > To: FFmpeg development discussions and patches > >

Re: [FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-29 Thread Hendrik Leppkes
On Thu, Jan 28, 2016 at 10:17 PM, Ari Koivula wrote: > I'm not quite sure if I'm going about this interlacing thing correctly, so > I figured I should send in a patch and ask. > > Is AV_CODEC_FLAG_INTERLACED_DCT the correct flag to look at? There is also > AV_CODEC_FLAG_INTERLACED_ME, but libx264

Re: [FFmpeg-devel] [PATCH]lavc/v210dec: Accept odd width

2016-01-29 Thread Michael Niedermayer
On Fri, Jan 29, 2016 at 11:20:18AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5195 for me. > > Please comment, Carl Eugen > v210dec.c |6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) > 9dcf44c7054d5443b80673b9717f22fa8c14c48b patchv210.diff > diff --gi

[FFmpeg-devel] Fwd: libavformat/segment : add option to increment timecode

2016-01-29 Thread Martin Vignali
Ping -- Forwarded message -- From: Martin Vignali Date: 2015-10-03 15:49 GMT+02:00 Subject: libavformat/segment : add option to increment timecode To: FFmpeg development discussions and patches Hello, in attach a patch, who add an option to increment timecode using segment. T

Re: [FFmpeg-devel] [PATCH] libkvazaar: add support for interlaced coding

2016-01-29 Thread Ari Koivula
On Thu, Jan 28, 2016 at 11:49 PM, Carl Eugen Hoyos wrote: > Ari Koivula koivu.la> writes: > > > Tell libkvazaar to use field coding when -flags +ildct > > is detected, resulting in two pictures being output for > > every frame instead of one. > > Is this really how it should be done? And is this

Re: [FFmpeg-devel] [PATCH] ffmpeg: extend -benchmark_all option to show elapsed time

2016-01-29 Thread Stefano Sabatini
On date Wednesday 2016-01-27 21:08:19 +0100, Michael Niedermayer encoded: > On Mon, Jan 11, 2016 at 06:21:15PM +0100, Stefano Sabatini wrote: > > The value is useful to get a raw estimate of the expected > > encoding/decoding time. > > > > Also extend option documentation. > > --- > > doc/ffmpeg.

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace "flush Media" with "flush_media" in benchmark_all output

2016-01-29 Thread Stefano Sabatini
On date Wednesday 2016-01-27 18:27:33 +0100, James Darnley encoded: > On 2016-01-27 13:09, Stefano Sabatini wrote: > > Simplify parsing and consistency. > > Fine. > > (Ha. It looks like I forgot to press send on this before going out.) Applied, thanks. -- FFmpeg = Fostering and Formidable Mult

Re: [FFmpeg-devel] mxfenc: Writes incomplete Video Line Map for progressive content

2016-01-29 Thread Carl Eugen Hoyos
Sebastian Dröge coaxion.net> writes: > Unless a later version of the standard somehow changes something > there, I would say that this commit should be reverted. Then please send a patch. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.o

Re: [FFmpeg-devel] mxfenc: Writes incomplete Video Line Map for progressive content

2016-01-29 Thread Sebastian Dröge
On Do, 2016-01-28 at 15:25 +0100, Sebastian Dröge wrote: > Hi everybody, > > since the following commit > https://git.videolan.org/?p=ffmpeg.git;a=commit;h=8ed82d8174a666f80ab8834e3617cbe91ae740a9 > mxfenc is only writing one value for progressive content. > Unfortunately the commit comes without

[FFmpeg-devel] [PATCH]lavc/v210dec: Accept odd width

2016-01-29 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #5195 for me. Please comment, Carl Eugen diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c index 12d5d54..9af9af6 100644 --- a/libavcodec/v210dec.c +++ b/libavcodec/v210dec.c @@ -53,10 +53,6 @@ static av_cold int decode_init(AVCodecContext *avctx) { V2

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libaacplus

2016-01-29 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > > > It's inferior to fdk-aac and has an arguably more > > > problematic license. > > > > Needs a Changelog entry and a news entry together with > > libvo. > What do you mean by news entry? On the website? Yes, a news entry on the website mentioning the removal

Re: [FFmpeg-devel] [PATCH 4/4] tests: Add test for proper header guard

2016-01-29 Thread Clément Bœsch
On Thu, Jan 28, 2016 at 06:33:56PM -0800, Timothy Gu wrote: > --- > tests/fate/source-check.sh | 11 +++ > tests/ref/fate/source | 12 > 2 files changed, 23 insertions(+) > > diff --git a/tests/fate/source-check.sh b/tests/fate/source-check.sh > index fb7af98..5524236 10