Re: [FFmpeg-devel] [PATCH] vf_overlay: Add `timebase` option to set the time base source for the output video.

2016-03-10 Thread Vittorio Gambaletta (VittGam)
On 10/03/2016 19:14:10 CET, Michael Niedermayer wrote: On Tue, Feb 09, 2016 at 09:12:53PM +0100, Paul B Mahol wrote: On 1/20/16, Vittorio Gambaletta (VittGam) wrote: > With this commit it is possible to use vf_overlay to center a video over > a black background, without having to kn

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

2016-01-26 Thread Vittorio Gambaletta (VittGam)
On 26/01/2016 23:05:22 CET, Neil Birkbeck wrote: Some sort of squash fail. Apologies. No problem! ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Hi, On 26/01/2016 22:09:02 CET, Neil Birkbeck wrote: From: "Vittorio Gambaletta (VittGam)" This is not from me... Cheers, Vittorio ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 2/2] ffmpeg_opt: Allow -metadata option to set metadata on programs.

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Hi, On 26/01/2016 17:37:56 CET, Michael Niedermayer wrote: On Tue, Jan 26, 2016 at 03:13:09PM +0100, Vittorio Gambaletta (VittGam) wrote: Signed-off-by: Vittorio Gambaletta --- doc/ffmpeg.texi |4 ++-- ffmpeg_opt.c|7 +++ 2 files changed, 9 insertions(+), 2 deletions

Re: [FFmpeg-devel] [PATCH] doc/filters: add an example to scale

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Hi, On 26/01/2016 15:20:22 CET, Vittorio Gambaletta (VittGam) wrote: Hi, On 26/01/2016 15:16:21 CET, James Darnley wrote: On 2016-01-22 14:44, Michael Niedermayer wrote: On Fri, Jan 22, 2016 at 03:53:10AM +0100, James Darnley wrote: Someone on IRC asked for a scale that would fit in a given

Re: [FFmpeg-devel] [PATCH] doc/filters: add an example to scale

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Hi, On 26/01/2016 15:16:21 CET, James Darnley wrote: On 2016-01-22 14:44, Michael Niedermayer wrote: On Fri, Jan 22, 2016 at 03:53:10AM +0100, James Darnley wrote: Someone on IRC asked for a scale that would fit in a given box. This is the answer. I couldn't see it in the existing examples s

[FFmpeg-devel] [PATCH v2 1/2] ffmpeg_opt: Move the 'process manually set programs' block above 'process manually set metadata' in open_output_file().

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Signed-off-by: Vittorio Gambaletta --- ffmpeg_opt.c | 112 +- 1 file changed, 56 insertions(+), 56 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 9b341cf..e03ad89 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -2358,6 +2358,6

[FFmpeg-devel] [PATCH v2 2/2] ffmpeg_opt: Allow -metadata option to set metadata on programs.

2016-01-26 Thread Vittorio Gambaletta (VittGam)
Signed-off-by: Vittorio Gambaletta --- doc/ffmpeg.texi |4 ++-- ffmpeg_opt.c|7 +++ 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index a38a32e..7d3266a 100644 --- a/doc/ffmpeg.texi +++ b/doc/ffmpeg.texi @@ -339,8 +339,8 @@ see @r

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: Fix multi program so that it supports adding the same stream to multiple programs.

2016-01-25 Thread Vittorio Gambaletta (VittGam)
Hi, On 25/01/2016 19:35:58 CET, Michael Niedermayer wrote: On Sun, Jan 24, 2016 at 03:54:34PM +0100, Vittorio Gambaletta (VittGam) wrote: Hi, On 24/01/2016 12:01:38 CET, Michael Niedermayer wrote: >On Sun, Jan 24, 2016 at 06:06:36AM +0100, Vittorio Gambaletta (VittGam) wrote: >>Sign

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: Allow program metadata other than title to be set.

2016-01-24 Thread Vittorio Gambaletta (VittGam)
Hi, On 24/01/2016 12:33:51 CET, Michael Niedermayer wrote: On Sun, Jan 24, 2016 at 05:23:22AM +0100, Vittorio Gambaletta (VittGam) wrote: This commit makes it possible to add optional metadata (such as "service_provider") to an mpegts output stream composed by multiple programs (mp

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: Fix multi program so that it supports adding the same stream to multiple programs.

2016-01-24 Thread Vittorio Gambaletta (VittGam)
Hi, On 24/01/2016 12:01:38 CET, Michael Niedermayer wrote: On Sun, Jan 24, 2016 at 06:06:36AM +0100, Vittorio Gambaletta (VittGam) wrote: Signed-off-by: Vittorio Gambaletta --- libavformat/mpegtsenc.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a

[FFmpeg-devel] [PATCH] avformat/mpegtsenc: Fix multi program so that it supports adding the same stream to multiple programs.

2016-01-23 Thread Vittorio Gambaletta (VittGam)
Signed-off-by: Vittorio Gambaletta --- libavformat/mpegtsenc.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 2c12043..a20e229 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -275

[FFmpeg-devel] [PATCH] ffmpeg_opt: Allow program metadata other than title to be set.

2016-01-23 Thread Vittorio Gambaletta (VittGam)
This commit makes it possible to add optional metadata (such as "service_provider") to an mpegts output stream composed by multiple programs (mpts). This is needed because the global metadata is not inherited by the single programs. Signed-off-by: Vittorio Gambaletta --- ffmpeg_opt.c | 10

[FFmpeg-devel] [PATCH] vf_overlay: Add `timebase` option to set the time base source for the output video.

2016-01-20 Thread Vittorio Gambaletta (VittGam)
With this commit it is possible to use vf_overlay to center a video over a black background, without having to know the video framerate in advance and set it as parameter to lavfi. Signed-off-by: Vittorio Gambaletta --- doc/filters.texi | 14 ++ libavfilter/vf_overlay.c |

[FFmpeg-devel] [PATCH] ffplay: Update docs after previous changes in ffplay mouse behaviour.

2016-01-19 Thread Vittorio Gambaletta (VittGam)
Signed-off-by: Vittorio Gambaletta --- doc/ffplay.texi |5 - ffplay.c|3 ++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 2a35c21..4bc3ced 100644 --- a/doc/ffplay.texi +++ b/doc/ffplay.texi @@ -238,9 +238,12 @@ Seek to

Re: [FFmpeg-devel] [PATCH v3 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-19 Thread Vittorio Gambaletta (VittGam)
On 19/01/2016 23:40:22 CET, Marton Balint wrote: On Tue, 19 Jan 2016, Marton Balint wrote: On Tue, 19 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse

[FFmpeg-devel] [PATCH v3 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-18 Thread Vittorio Gambaletta (VittGam)
Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. Signed-off-by: Vittorio Gambaletta --- Changelog |1 + ffplay.c | 10 ++ 2 files changed, 11 insertions(+

[FFmpeg-devel] [PATCH v3 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-18 Thread Vittorio Gambaletta (VittGam)
Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse button is used to click and drag on the window. Signed-off-b

Re: [FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
ttorio On 16/01/2016 22:41:30 CET, Vittorio Gambaletta (VittGam) wrote: Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when

[FFmpeg-devel] [PATCH v2 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. Signed-off-by: Vittorio Gambaletta --- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay

[FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse button is used to click and drag on the window. Signed-off-b

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Hi, On 16/01/2016 21:25:00 CET, Marton Balint wrote: On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. I am

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Hi, On 16/01/2016 21:13:14 CET, Marton Balint wrote: On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit

[FFmpeg-devel] [PATCH 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-15 Thread Vittorio Gambaletta (VittGam)
Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. Signed-off-by: Vittorio Gambaletta --- ffplay.c | 18 ++ 1 file changed, 18 insertions(+) diff --git

[FFmpeg-devel] [PATCH 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-15 Thread Vittorio Gambaletta (VittGam)
Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse button is used to click and drag on the window. Signed-off-

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
On 15/01/2016 00:09:37 CET, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 6:05 PM, Marton Balint wrote: [..] I think it would make sense to change ffplay so that only the right mouse button would seek. This way we can avoid the issue. Anybody against this? Kind of, it is slightly easier t

Re: [FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
Hi, On 14/01/2016 23:15:16 CET, Ganesh Ajjanagadde wrote: On Thu, Jan 14, 2016 at 3:56 PM, Vittorio Gambaletta (VittGam) wrote: --- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay.c b/ffplay.c index d2e3dc6..0837d58 100644 --- a/ffplay.c +++ b/ffplay.c

[FFmpeg-devel] [PATCH] ffplay: Add ignore_keydown and ignore_mousedown options.

2016-01-14 Thread Vittorio Gambaletta (VittGam)
--- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay.c b/ffplay.c index d2e3dc6..0837d58 100644 --- a/ffplay.c +++ b/ffplay.c @@ -336,6 +336,8 @@ static int decoder_reorder_pts = -1; static int autoexit; static int exit_on_keydown; static int exit_on_mousedown;

Re: [FFmpeg-devel] [PATCH] Revert "matroskaenc: refuse to write AAC without valid extradata"

2015-09-21 Thread Vittorio Gambaletta (VittGam)
On 21/09/2015 12:15:48 CEST, Hendrik Leppkes wrote: AAC without extradata is invalid, there is no discussion about that. As wm4 says, just because ffmpeg accepts it anyway doesn't really change that. Ok. So, how can I check that the resulting file actually has missing AAC extradata, despite ff

Re: [FFmpeg-devel] [PATCH] Revert "matroskaenc: refuse to write AAC without valid extradata"

2015-09-21 Thread Vittorio Gambaletta (VittGam)
Il 21.09.2015 07:39:24 Ronald S. Bultje ha scritto: Is this still necessary if you use the adtstoaasc bitstream filter? Yes. Il 21.09.2015 09:45:55 Hendrik Leppkes ha scritto: Simply reverting and thus allowing broken files to be created once again is not a good idea. I recommend to properly