[FFmpeg-devel] [PATCH] dashenc: replace attribute id with contentType for the AdaptationSet element

2015-05-09 Thread James Almer
id should be an integer, not a string. It is also optional, so use contentType instead which is the proper attribute for these values. This addresses ticket #4545, fixing an MPD validation error. Signed-off-by: James Almer --- libavformat/dashenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 d

Re: [FFmpeg-devel] [PATCH] avformat_alloc_context: initialize raw_packet_buffer_remaining_size.

2015-05-09 Thread Michael Niedermayer
On Sat, May 09, 2015 at 08:57:26PM +0200, Reimar Döffinger wrote: > This currently works for most users because > avformat_open_input sets it, but this patch fixes any > applications not using that function. > > Signed-off-by: Reimar Döffinger > --- > libavformat/options.c | 1 + > 1 file change

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread wm4
On Sat, 9 May 2015 21:30:33 +0200 Nicolas George wrote: > Le decadi 20 floréal, an CCXXIII, wm4 a écrit : > > Uh, I replied with what exactly you asked for. > > Carl Eugen asked for alternate wordings, and instead of trying honestly to > propose something that would be acceptable, you propose so

Re: [FFmpeg-devel] [PATCH] Add changelog entry for profile 1/2/3 addition to native VP9 decoder.

2015-05-09 Thread Ronald S. Bultje
Hi, On Sat, May 9, 2015 at 2:33 PM, James Almer wrote: > On 09/05/15 3:31 PM, Ronald S. Bultje wrote: > > --- > > Changelog | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Changelog b/Changelog > > index 7d619b9..e261395 100644 > > --- a/Changelog > > +++ b/Changelog > > @@ -19,6

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Nicolas George
Le decadi 20 floréal, an CCXXIII, wm4 a écrit : > Uh, I replied with what exactly you asked for. Carl Eugen asked for alternate wordings, and instead of trying honestly to propose something that would be acceptable, you propose something that is deliberately unacceptable. In other words, you are

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread wm4
On Sat, 9 May 2015 21:15:14 +0200 Nicolas George wrote: > Le decadi 20 floréal, an CCXXIII, wm4 a écrit : > > --demuxer-lavf-o=use_absolute_path=1 > > So basically, you whine about a dash but you can not be bothered to compose > a message that does not have sixteen app-specific characters. Uh,

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Nicolas George
Le decadi 20 floréal, an CCXXIII, wm4 a écrit : > --demuxer-lavf-o=use_absolute_path=1 So basically, you whine about a dash but you can not be bothered to compose a message that does not have sixteen app-specific characters. Regards, -- Nicolas George signature.asc Description: Digital sign

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread wm4
On Sat, 9 May 2015 20:49:23 +0200 Nicolas George wrote: > Carl Eugen: > > > > > > + "Absolute path %s not tried for security reasons, " > > > > > > + "pass -use_absolute_path 1 to force using absolute paths\n", > > "Enable the use_absolute_path option to allow using absolute paths" >

[FFmpeg-devel] [PATCH] avformat_alloc_context: initialize raw_packet_buffer_remaining_size.

2015-05-09 Thread Reimar Döffinger
This currently works for most users because avformat_open_input sets it, but this patch fixes any applications not using that function. Signed-off-by: Reimar Döffinger --- libavformat/options.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/options.c b/libavformat/options.c inde

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Nicolas George
Carl Eugen: > > > > > + "Absolute path %s not tried for security reasons, " > > > > > + "pass -use_absolute_path 1 to force using absolute paths\n", "Enable the use_absolute_path option to allow using absolute paths" wm4: > I suspect you would reject the wording needed for my own app

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread wm4
On Sat, 9 May 2015 18:35:44 + (UTC) Carl Eugen Hoyos wrote: > Carl Eugen Hoyos ag.or.at> writes: > > > > > + "Absolute path %s not tried for security reasons, " > > > > + "pass -use_absolute_path 1 to force using absolute paths\n", > > > Please suggest a wording that is also ac

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > > > + "Absolute path %s not tried for security reasons, " > > > + "pass -use_absolute_path 1 to force using absolute paths\n", > Please suggest a wording that is also acceptable for > library users (they can't guess "use_absolute_path" > either

Re: [FFmpeg-devel] [PATCH] Add changelog entry for profile 1/2/3 addition to native VP9 decoder.

2015-05-09 Thread James Almer
On 09/05/15 3:31 PM, Ronald S. Bultje wrote: > --- > Changelog | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Changelog b/Changelog > index 7d619b9..e261395 100644 > --- a/Changelog > +++ b/Changelog > @@ -19,6 +19,7 @@ version : > - chorus filter > - Canopus HQ/HQA decoder > - Autom

[FFmpeg-devel] [PATCH] Add changelog entry for profile 1/2/3 addition to native VP9 decoder.

2015-05-09 Thread Ronald S. Bultje
--- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index 7d619b9..e261395 100644 --- a/Changelog +++ b/Changelog @@ -19,6 +19,7 @@ version : - chorus filter - Canopus HQ/HQA decoder - Automatically rotate videos based on metadata in ffmpeg +- profile 1 (4:2:

Re: [FFmpeg-devel] [PATCH 3/3] configure: loongson enable subarch mips64

2015-05-09 Thread Michael Niedermayer
On Fri, May 08, 2015 at 06:17:59PM +0800, 周晓勇 wrote: > From 1094ca8983b34492dece7da5cd09125dbda04726 Mon Sep 17 00:00:00 2001 > From: ZhouXiaoyong > Date: Fri, 8 May 2015 09:08:00 +0800 > Subject: [PATCH 3/3] configure: loongson enable subarch mips64 > > 1.Fedora19(Loongson) has released 32bit ve

Re: [FFmpeg-devel] [PATCH 2/3] configure: remove loongson check inline asm and mips dependent

2015-05-09 Thread 周晓勇
> > From 6f76746ecd5b623e7610e293be2cbe02022dd22a Mon Sep 17 00:00:00 2001 > > From: ZhouXiaoyong > > Date: Thu, 7 May 2015 11:50:42 +0800 > > Subject: [PATCH 2/3] configure: remove loongson check inline asm and mips > > dependent > > > > 1.Loongson now have two series CPU supported MMI(Multi-Med

Re: [FFmpeg-devel] [PATCH 3/3] configure: loongson enable subarch mips64

2015-05-09 Thread 周晓勇
> On Fri, May 08, 2015 at 06:17:59PM +0800, 周晓勇 wrote: > > From 1094ca8983b34492dece7da5cd09125dbda04726 Mon Sep 17 00:00:00 2001 > > From: ZhouXiaoyong > > Date: Fri, 8 May 2015 09:08:00 +0800 > > Subject: [PATCH 3/3] configure: loongson enable subarch mips64 > > > > 1.Fedora19(Loongson) has rel

Re: [FFmpeg-devel] [PATCH]lavf/wavdec: Allow decoding of invalid large wav files

2015-05-09 Thread Michael Niedermayer
On Fri, May 08, 2015 at 08:01:15PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Valid wav files are not larger than 4G, files that are > larger are truncated by current FFmpeg. > Attached patch allows reading such files if the size > was (imo "correctly") written to -1. > > Please comment, Carl Eug

Re: [FFmpeg-devel] [PATCH]Print an error for invalid wav files and write no incorrect file length

2015-05-09 Thread Michael Niedermayer
On Fri, May 08, 2015 at 08:23:35PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4543 for me. > > Please comment, Carl Eugen > wavenc.c |8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > 7e2f589f1d543161f5b7d3682d7cd02cf76ff74e patchwav4genc.diff prob

Re: [FFmpeg-devel] [PATCH]Force colour range for vc1 gray output

2015-05-09 Thread Michael Niedermayer
On Sat, May 09, 2015 at 12:32:03PM +0200, Carl Eugen Hoyos wrote: > On Saturday 09 May 2015 11:00:02 am Carl Eugen Hoyos wrote: > > Hi! > > > > I believe attached patch makes vc1 gray output more correct. > > New patch attached that allows users to set the range. > > Carl Eugen > vc1dec.c |

Re: [FFmpeg-devel] [PATCH]Force tv range for mpeg4 gray output

2015-05-09 Thread Michael Niedermayer
On Sat, May 09, 2015 at 12:30:32PM +0200, Carl Eugen Hoyos wrote: [...] > @@ -48,8 +48,11 @@ static enum AVPixelFormat h263_get_format(AVCodecContext > *avctx) > if (avctx->codec->id == AV_CODEC_ID_MSS2) > return AV_PIX_FMT_YUV420P; > > -if (CONFIG_GRAY && (avctx->flags & CODEC

Re: [FFmpeg-devel] [PATCH]Force colour range for vc1 gray output

2015-05-09 Thread Carl Eugen Hoyos
On Saturday 09 May 2015 11:00:02 am Carl Eugen Hoyos wrote: > Hi! > > I believe attached patch makes vc1 gray output more correct. New patch attached that allows users to set the range. Carl Eugen diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 86e214a..ad1d4d9 100644 --- a/libavcode

Re: [FFmpeg-devel] [PATCH]Force tv range for mpeg4 gray output

2015-05-09 Thread wm4
On Sat, 9 May 2015 12:30:32 +0200 Carl Eugen Hoyos wrote: > On Saturday 09 May 2015 11:53:13 am wm4 wrote: > > On Sat, 9 May 2015 11:12:46 +0200 > > > > Carl Eugen Hoyos wrote: > > > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c > > > index c47f5b1..162c295 100644 > > > --- a/libavcod

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Carl Eugen Hoyos
wm4 googlemail.com> writes: > What I'm suggesting would fix all of this. But how is your suggestion related to the patch in question? Please suggest a wording that is less ffmpeg-related, I will happily use it. Carl Eugen ___ ffmpeg-devel mailing li

Re: [FFmpeg-devel] [PATCH]Force tv range for mpeg4 gray output

2015-05-09 Thread Carl Eugen Hoyos
On Saturday 09 May 2015 11:53:13 am wm4 wrote: > On Sat, 9 May 2015 11:12:46 +0200 > > Carl Eugen Hoyos wrote: > > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c > > index c47f5b1..162c295 100644 > > --- a/libavcodec/h263dec.c > > +++ b/libavcodec/h263dec.c > > @@ -48,8 +48,10 @@ static

Re: [FFmpeg-devel] [PATCH]Force colour range for vc1 gray output

2015-05-09 Thread Michael Niedermayer
On Sat, May 09, 2015 at 11:00:02AM +0200, Carl Eugen Hoyos wrote: > Hi! > > I believe attached patch makes vc1 gray output more correct. > > Or should this be set unconditionally for all output pix_fmts? > > Please comment, Carl Eugen > vc1dec.c |4 +++- > 1 file changed, 3 insertions(+),

Re: [FFmpeg-devel] [PATCH]Force tv range for mpeg4 gray output

2015-05-09 Thread wm4
On Sat, 9 May 2015 11:12:46 +0200 Carl Eugen Hoyos wrote: > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c > index c47f5b1..162c295 100644 > --- a/libavcodec/h263dec.c > +++ b/libavcodec/h263dec.c > @@ -48,8 +48,10 @@ static enum AVPixelFormat h263_get_format(AVCodecContext > *avctx) >

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread wm4
On Sat, 9 May 2015 10:19:40 +0200 Nicolas George wrote: > Le nonidi 19 floréal, an CCXXIII, wm4 a écrit : > > The problem is that this code was written with ffmpeg.c in mind, so you > > assumed that it was fine to open new files inside of libavformat, and > > you had to add an option to let ffmpe

[FFmpeg-devel] [PATCH]Force tv range for mpeg4 gray output

2015-05-09 Thread Carl Eugen Hoyos
Hi! Attached patch fixes the colour range for mpeg4 gray decoding. Less ugly suggestions welcome. Please comment, Carl Eugen diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index c47f5b1..162c295 100644 --- a/libavcodec/h263dec.c +++ b/libavcodec/h263dec.c @@ -48,8 +48,10 @@ static enum

[FFmpeg-devel] [PATCH]Force colour range for vc1 gray output

2015-05-09 Thread Carl Eugen Hoyos
Hi! I believe attached patch makes vc1 gray output more correct. Or should this be set unconditionally for all output pix_fmts? Please comment, Carl Eugen diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c index 86e214a..1a9e596 100644 --- a/libavcodec/vc1dec.c +++ b/libavcodec/vc1dec.c @@ -

Re: [FFmpeg-devel] [PATCH]Tell users about -use_absolute_path

2015-05-09 Thread Nicolas George
Le nonidi 19 floréal, an CCXXIII, wm4 a écrit : > The problem is that this code was written with ffmpeg.c in mind, so you > assumed that it was fine to open new files inside of libavformat, and > you had to add an option to let ffmpeg.c "fix" the hardcoded policy if > needed. Not at all. "use_abso