Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Matt Oliver
> > When running ffmpeg on Debian (main), where these Nvidia blobs > are not present and not needed (because nouveau is used as driver), > no component of the operating system, major or not, > has been distributed with Nvidia's blobs, so the system library > exception does not apply. > > I'm not

Re: [FFmpeg-devel] Question about av_frame_is_writable

2015-12-12 Thread Paul B Mahol
On 12/12/15, Jean Delvare wrote: > Hi all, > > Can av_frame_is_writable() ever return 1, and if so, when? > > Context: I am testing corner cases of the delogo filter. To my > surprise, even for a simple test case such as: > > $ ffmpeg -f lavfi -i "color=color=white:size=24x24"

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Philip Langdale
On 2015-12-12 20:33, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Philip Langdale a écrit : it would be highly illogical to conclude that section 6/7 do not apply to the original code itself and that we need to construct a separate entity that does the combination for it to be

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > I don't agree as I have already explained previously. Fortunately, your arguments have no legal standing. > Have a look at what some of the proprietary licenses require for distribution > that has absolutely nothing to do with how

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > However, the (L)GPL has a rule that allows distribution of a > binary only together with the complete corresponding source code, > that also includes the source code of any dynamically loaded > library needed at run-time. No, it does

[FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
I am happy to announce that I will now be using Alpine instead of the Yahoo website for posting to this mailing list, since the accepted policy here seems to be bottom-posting only. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > Is in a previous mail. I looked and did not see it. > I'm not continuing this silly conversation. > You can look up the meaning of words in a dictionary. This has nothing silly. Having the code of a codec interpret the bitstream of

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : Now, let's not split hairs. Be happy that I have stopped the top-posting instead. Not that I care a lot what you think. It has not taken you a long time to show your true colors here too.

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 13:27, Philip Langdale wrote: > On 2015-12-12 19:26, Andreas Cadhalpun wrote: >> Anyway, I think your idea about a license exception would be a good solution. >> We could add a special exception to nvenc.c, allowing its use with Nvidia's >> blobs, something like [1]: >> >> In

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 12:55, Nicolas George wrote: > Le primidi 21 frimaire, an CCXXIV, Carl Eugen Hoyos a écrit : >> I am glad we agree that there is no difference (license-wise) if >> a library is linked statically, dynamically or via dynamic >> loading;-) > > You are, unfortunately, completely

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : > Oh, such a lovely reply. Too bad you did only take a very small part into account. What did you not understand in "trimmed interleaved posting"? Regards, -- Nicolas George signature.asc Description: Digital signature

[FFmpeg-devel] Question about av_frame_is_writable

2015-12-12 Thread Jean Delvare
Hi all, Can av_frame_is_writable() ever return 1, and if so, when? Context: I am testing corner cases of the delogo filter. To my surprise, even for a simple test case such as: $ ffmpeg -f lavfi -i "color=color=white:size=24x24" -frames:v 1 -vf "delogo=7:3:8:8" -f image2 debug.png

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 10:37, Matt Oliver wrote: >> >> When running ffmpeg on Debian (main), where these Nvidia blobs >> are not present and not needed (because nouveau is used as driver), >> no component of the operating system, major or not, >> has been distributed with Nvidia's blobs, so the system

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/aacsbr: Split the env_facs table

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 05:14:55PM +0100, Andreas Cadhalpun wrote: > On 19.11.2015 14:17, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > This also removes a #ifdef and special case for the fixed point case > > > > Signed-off-by: Michael Niedermayer

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 11:35, Hendrik Leppkes wrote: > On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun > wrote: >> On 12.12.2015 10:50, Hendrik Leppkes wrote: >>> We should just add an exception into the license to explicitly allow >>> using it with the NVIDIA CUDA

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le primidi 21 frimaire, an CCXXIV, Carl Eugen Hoyos a écrit : > I am glad we agree that there is no difference (license-wise) if > a library is linked statically, dynamically or via dynamic > loading;-) You are, unfortunately, completely wrong. It makes all the difference in the world. Static

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Philip Langdale
On 2015-12-12 19:26, Andreas Cadhalpun wrote: On 12.12.2015 11:35, Hendrik Leppkes wrote: On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun wrote: On 12.12.2015 10:50, Hendrik Leppkes wrote: We should just add an exception into the license to explicitly

Re: [FFmpeg-devel] Question about av_frame_is_writable

2015-12-12 Thread Jean Delvare
Hi Paul, On Sat, 12 Dec 2015 12:03:39 +, Paul B Mahol wrote: > On 12/12/15, Jean Delvare wrote: > > Can av_frame_is_writable() ever return 1, and if so, when? > > > > Context: I am testing corner cases of the delogo filter. To my > > surprise, even for a simple test case

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : I am happy to announce that I will now be using Alpine instead of the Yahoo I must admit, I do not care very much what you do with your life. Would you also tell us what you ate for breakfast?

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : > It does. [citation needed] > Multimedia files can't be executed, only decoded. Define the difference. Regards, -- Nicolas George signature.asc Description: Digital signature ___

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Mats Peterson wrote: On Sat, 12 Dec 2015, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : Oh, such a lovely reply. Too bad you did only take a very small part into account. What did you not understand in "trimmed interleaved posting"?

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 10:50, Hendrik Leppkes wrote: > On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun > wrote: >> On 12.12.2015 01:46, Philip Langdale wrote: >>> On 2015-12-12 00:03, Andreas Cadhalpun wrote: On 11.12.2015 09:41, Carl Eugen Hoyos wrote: > My

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : Oh, such a lovely reply. Too bad you did only take a very small part into account. What did you not understand in "trimmed interleaved posting"? Regards, -- Nicolas George Now, let's

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:42, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> It does. > > [citation needed] Is in a previous mail. >> Multimedia files can't be executed, only decoded. > > Define the difference. I'm not continuing this silly conversation. You

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 01:46, Philip Langdale wrote: > On 2015-12-12 00:03, Andreas Cadhalpun wrote: >> On 11.12.2015 09:41, Carl Eugen Hoyos wrote: >>> My point is that so far several people have said that if nvenc >>> is a system library there is no issue (and I fully agree). I >>> didn't see a mail (and

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Hendrik Leppkes
On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun wrote: > On 12.12.2015 01:46, Philip Langdale wrote: >> On 2015-12-12 00:03, Andreas Cadhalpun wrote: >>> On 11.12.2015 09:41, Carl Eugen Hoyos wrote: My point is that so far several people have said that

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 11:32:13AM +, Mats Peterson wrote: > OK, here is an improved version of my previous patch of matroskadec.c that > manages the palette itself without calling code in mov.c. I have borrowed > some code from it, though, and I'm including qtpalette.h. One thing I don't >

Re: [FFmpeg-devel] [PATCH RFC] avfilter/vf_delogo: Use AVPixFmtDescriptor.nb_components

2015-12-12 Thread Paul B Mahol
On 12/10/15, Jean Delvare wrote: > Relying on AVPixFmtDescriptor.nb_components is cleaner and faster than > checking data and linesize for every possible plane. > > Signed-off-by: Jean Delvare > --- > I see that most filters use

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Philip Langdale a écrit : > it would be highly illogical to conclude that section 6/7 do not apply to > the original code itself and that we need to construct a separate entity > that does the combination for it to be licence compliant. It would be fairly

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:29, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> However, the (L)GPL has a rule that allows distribution of a >> binary only together with the complete corresponding source code, >> that also includes the source code of any dynamically

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Hendrik Leppkes
On Sat, Dec 12, 2015 at 11:25 AM, Andreas Cadhalpun wrote: > On 12.12.2015 10:50, Hendrik Leppkes wrote: >> On Sat, Dec 12, 2015 at 10:39 AM, Andreas Cadhalpun >> wrote: >>> On 12.12.2015 01:46, Philip Langdale wrote: On

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Mats Peterson
21 is one odd number, to start with. It will make the privata end in the middle of the vendor fourcc, which is often 'appl'. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ - Original Message - From: Mats Peterson To: FFmpeg development

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Mats Peterson
Sorry for my typos. "Privata" really means "private data" ;) Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ - Original Message - From: Mats Peterson To: FFmpeg development discussions and patches Sent: Saturday,

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-12 Thread Andreas Cadhalpun
On 12.12.2015 14:03, Nicolas George wrote: > Le duodi 22 frimaire, an CCXXIV, Andreas Cadhalpun a écrit : >> I don't agree as I have already explained previously. > > Fortunately, your arguments have no legal standing. Neither have yours. > The (L)GPL does not have a rule against distributing a

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : > I am happy to announce that I will now be using Alpine instead of the Yahoo I must admit, I do not care very much what you do with your life. Would you also tell us what you ate for breakfast? > website for posting to this mailing list,

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Nicolas George
Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : > >Now, let's not split hairs. Be happy that I have stopped the top-posting > >instead. > Not that I care a lot what you think. It has not taken you a long time to show your true colors here too. For the record, this was the guy who

Re: [FFmpeg-devel] [PATCH RFC] avfilter/vf_delogo: Use AVPixFmtDescriptor.nb_components

2015-12-12 Thread Michael Niedermayer
On Sat, Dec 12, 2015 at 12:07:25PM +, Paul B Mahol wrote: > On 12/10/15, Jean Delvare wrote: > > Relying on AVPixFmtDescriptor.nb_components is cleaner and faster than > > checking data and linesize for every possible plane. > > > > Signed-off-by: Jean Delvare

Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

2015-12-12 Thread Carl Eugen Hoyos
On Friday 11 December 2015 05:04:21 pm Michael Niedermayer wrote: > > matroskadec.c |2 ++ > > 1 file changed, 2 insertions(+) > > 77e6700373ba86f49886c58ef28a9ffbd9b7e001 patchmkvaudiofourcc.diff > > diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c > > index

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 12:09:57PM -0500, Ganesh Ajjanagadde wrote: > On Fri, Dec 11, 2015 at 11:36 AM, Andreas Cadhalpun > wrote: > > On 11.12.2015 17:21, Ganesh Ajjanagadde wrote: > >> On Fri, Dec 11, 2015 at 11:16 AM, Andreas Cadhalpun > >>

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Ganesh Ajjanagadde
On Sat, Dec 12, 2015 at 10:19 AM, Mats Peterson wrote: [...] > Even with the first patch, there isn't a lot to be technically bad about, > since it concerns something as simple as putting the palette in extradata. > Calling a function in mov.c could perhaps be

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Ganesh Ajjanagadde wrote: On Sat, Dec 12, 2015 at 10:19 AM, Mats Peterson wrote: [...] Even with the first patch, there isn't a lot to be technically bad about, since it concerns something as simple as putting the palette in extradata.

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Nicolas George wrote: Since the patches are technically bad too, I will ignore these mails from now on. Regards, -- Nicolas George Even with the first patch, there isn't a lot to be technically bad about, since it concerns something as simple as putting the palette

Re: [FFmpeg-devel] [PATCH 2/2] lavu/error: add missing error messages for errors supported on all platforms

2015-12-12 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 10:12:52PM +0100, Marton Balint wrote: > We need these if we have no strerror_r. > > Descriptions are taken from doc/errno.txt except for ENOMEM. > > Signed-off-by: Marton Balint > --- > libavutil/error.c | 37 + > 1

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/aacsbr_template: Add Check to read_sbr_envelope()

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 05:23:58PM +0100, Andreas Cadhalpun wrote: > On 19.11.2015 14:17, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > The limit is a conservative guess, the spec does not seem to specify a limit > > > > Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

2015-12-12 Thread Dan Sanders
I took a look at the structure of this file, and the encoding appears to be correct. The frame order in the moof boxes matches the H.264 stream, and the durations of the moof boxes match with the sidx. Some notes to help with analysis: There are no per-frame durations specified, every frame is

Re: [FFmpeg-devel] End of the top-posting

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Mats Peterson wrote: On Sat, 12 Dec 2015, Nicolas George wrote: Le duodi 22 frimaire, an CCXXIV, Mats Peterson a écrit : Now, let's not split hairs. Be happy that I have stopped the top-posting instead. Not that I care a lot what you think. It has not taken you a long

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

2015-12-12 Thread Ganesh Ajjanagadde
On Sat, Dec 12, 2015 at 12:58 PM, Michael Niedermayer wrote: > On Fri, Dec 11, 2015 at 12:09:57PM -0500, Ganesh Ajjanagadde wrote: >> On Fri, Dec 11, 2015 at 11:36 AM, Andreas Cadhalpun >> wrote: >> > On 11.12.2015 17:21, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

2015-12-12 Thread Michael Niedermayer
On Sat, Dec 12, 2015 at 01:08:01PM -0500, Ganesh Ajjanagadde wrote: > On Sat, Dec 12, 2015 at 12:58 PM, Michael Niedermayer > wrote: > > On Fri, Dec 11, 2015 at 12:09:57PM -0500, Ganesh Ajjanagadde wrote: > >> On Fri, Dec 11, 2015 at 11:36 AM, Andreas Cadhalpun > >>

Re: [FFmpeg-devel] [PATCH] avcodec/golomb: Mask shift amount before use in get_ue_golomb()

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 04:24:43PM -0500, Ganesh Ajjanagadde wrote: > On Fri, Dec 11, 2015 at 4:14 PM, Andreas Cadhalpun > wrote: > > On 07.12.2015 00:27, Ganesh Ajjanagadde wrote: > >> On Sun, Dec 6, 2015 at 6:12 PM, Andreas Cadhalpun > >>

Re: [FFmpeg-devel] [PATCH] avcodec/golomb: Mask shift amount before use in get_ue_golomb()

2015-12-12 Thread Ganesh Ajjanagadde
On Sat, Dec 12, 2015 at 1:13 PM, Michael Niedermayer wrote: [...] > btw, if you feel something should be backported, dont hesitate to > push cherry picked commits to the release branches > (with cherry-pick -x) > if you are unsure if somethig should be cherry picked, dont

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Michael Niedermayer
On Sat, Dec 12, 2015 at 11:17:00AM +, Mats Peterson wrote: > Obviously that private data is cropped in some way then, since the minimum > size of a video sample description in QuickTime video is 86 bytes. FFmpeg tries to support all kind of odd and broken files, so even if the file is

Re: [FFmpeg-devel] [PATCH v3 06/14] mips: i6400 core does have an FPU, so not disable it

2015-12-12 Thread Michael Niedermayer
On Thu, Dec 03, 2015 at 01:22:21PM +, Vicente Olivert Riera wrote: > Signed-off-by: Vicente Olivert Riera > --- > Changes v2 -> v3: > - Nothing. > > Changes v1 -> v2: > - Fix a grammar mistake in the commit log. Change "do has" by "does >have". Thanks to

Re: [FFmpeg-devel] [PATCH v3 04/14] mips: remove --mips64r6 configure option

2015-12-12 Thread Michael Niedermayer
On Thu, Dec 03, 2015 at 01:22:19PM +, Vicente Olivert Riera wrote: > Having a configure option with the same name as a MIPS ISA is confusing, > so better to remove it. This option was being used to add some > optimizations to a specific core (i6400). We will add the optimizations > just when

Re: [FFmpeg-devel] [PATCH v3 05/14] mips: add i6400 core optimizations

2015-12-12 Thread Michael Niedermayer
On Thu, Dec 03, 2015 at 01:22:20PM +, Vicente Olivert Riera wrote: > Signed-off-by: Vicente Olivert Riera > --- > Changes v2 -> v3: > - Nothing. > > Changes v1 -> v2: > - Nothing. > > configure | 7 --- > 1 file changed, 4 insertions(+), 3 deletions(-)

Re: [FFmpeg-devel] mpegtsenc cbr mode pcr accuracy

2015-12-12 Thread Michael Niedermayer
On Sat, Nov 28, 2015 at 09:03:20AM +0200, Timo Teras wrote: > Hi, > > While the pcr is accuracte, I was measuring the cbr pcr frequence with > opencaster suite's tspcmeasure tool, and it says that the frequency is > not that accurate. With default options (just muxrate specified) the > pcr should

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable parser for mp3s by old HandBrake

2015-12-12 Thread Michael Niedermayer
On Thu, Dec 10, 2015 at 01:42:52PM +0100, Michael Niedermayer wrote: > On Thu, Dec 10, 2015 at 10:37:42AM +0100, Moritz Barsnick wrote: > > On Wed, Dec 09, 2015 at 22:24:08 +0100, Michael Niedermayer wrote: > > > +if (sscanf(str, "HandBrake %d.%d.%d", , , > > > ) == 3) { > > > +

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Fix integer overflow in length computation

2015-12-12 Thread Michael Niedermayer
On Wed, Dec 09, 2015 at 06:13:25PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes: CID1341577 > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfenc.c |4 ++-- > 1 file changed, 2 insertions(+), 2

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Paul B Mahol wrote: On 12/12/15, Mats Peterson wrote: On Sat, 12 Dec 2015, Michael Niedermayer wrote: On Sat, Dec 12, 2015 at 11:17:00AM +, Mats Peterson wrote: Obviously that private data is cropped in some way then, since the

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Paul B Mahol
On 12/12/15, Mats Peterson wrote: > On Sat, 12 Dec 2015, Michael Niedermayer wrote: > >> On Sat, Dec 12, 2015 at 11:17:00AM +, Mats Peterson wrote: >>> Obviously that private data is cropped in some way then, since the >>> minimum size of a video sample

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add AVVideoFramePool

2015-12-12 Thread Michael Niedermayer
On Fri, Dec 11, 2015 at 07:35:08PM +0100, Matthieu Bouron wrote: > On Fri, Dec 11, 2015 at 7:07 PM, Michael Niedermayer > wrote: > > > On Fri, Dec 11, 2015 at 01:32:46PM +0100, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > > --- >

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

2015-12-12 Thread Mats Peterson
On Sat, 12 Dec 2015, Michael Niedermayer wrote: On Sat, Dec 12, 2015 at 11:17:00AM +, Mats Peterson wrote: Obviously that private data is cropped in some way then, since the minimum size of a video sample description in QuickTime video is 86 bytes. FFmpeg tries to support all kind of

Re: [FFmpeg-devel] Memory leaks in libavformat/segment.c

2015-12-12 Thread Michael Niedermayer
On Tue, Dec 08, 2015 at 07:29:13PM +0800, Lvqier wrote: > On 12/8/15 7:05 PM, Hendrik Leppkes wrote: > >On Tue, Dec 8, 2015 at 11:46 AM, Lvqier wrote: > >>On 12/8/15 6:26 PM, Hendrik Leppkes wrote: > >>>On Tue, Dec 8, 2015 at 11:19 AM, Lvqier wrote: > Hi

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

2015-12-12 Thread Ganesh Ajjanagadde
On Sat, Dec 12, 2015 at 1:17 PM, Michael Niedermayer wrote: [...] >> >> > >> >> > The exp2f expressions are: >> >> > exp2f(sbr->data[0].env_facs_q[e][k] * alpha + 7.0f); >> >> > exp2f((pan_offset - sbr->data[1].env_facs_q[e][k]) * alpha); >> >> > exp2f(NOISE_FLOOR_OFFSET -

Re: [FFmpeg-devel] [PATCH v4] Added VideoToolbox H.264 encoder.

2015-12-12 Thread Richard Kern
On Nov 28, 2015, at 5:23 PM, Richard Kern wrote: > > On Nov 28, 2015, at 5:07 PM, Timothy Gu wrote: >> >> On Sat, Nov 28, 2015 at 02:06:07PM -0500, Rick Kern wrote: >>> vtenc.c was updated to match the coding style of the project. >>> >>> I used

[FFmpeg-devel] mpegtsenc: signalling of AC-3 and E-AC-3 according to ATSC/DVB

2015-12-12 Thread Stefan Pöschel
[sorry, used wrong sender previously] Hi, within the MPEG-TS muxer, the signalling of AC-3 is done according to System A (ATSC) while in contrast E-AC-3 is signalled according to System B (DVB). To fix this inconsistency, E-AC-3 should IMO be signalled according to System A, too. Furthermore a

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

2015-12-12 Thread Michael Niedermayer
On Sat, Dec 12, 2015 at 05:24:34PM -0500, Ganesh Ajjanagadde wrote: > On Sat, Dec 12, 2015 at 1:17 PM, Michael Niedermayer wrote: > [...] > > >> >> > > >> >> > The exp2f expressions are: > >> >> > exp2f(sbr->data[0].env_facs_q[e][k] * alpha + 7.0f); > >> >> > exp2f((pan_offset