Re: [FFmpeg-devel] Set AV_TIMECODE_FLAG_ALLOWNEGATIVE in avutil/timecode.c av_timecode_init_from_string

2014-09-11 Thread Clément Bœsch
On Tue, Sep 09, 2014 at 12:24:21PM -0700, jon wrote: > If the timecode value is negative it certainly seems safe to set the > AV_TIMECODE_FLAG_ALLOWNEGATIVE on the AVTimecode object. > From 35ae38f9557e1ff01040d0e681e843d1727f979b Mon Sep 17 00:00:00 2001 > From: Jon Morley > Date: Tue, 9 Sep 201

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_pixel_formats option (was: add -show_pixel_descriptions option)

2014-09-11 Thread Tobias Rapp
Have now added documentation and cleaned up some unused variables with the new patch. Also on a second thought the option names "show_pixel_format_descriptions" or "show_pixel_formats" seem less confusing to me than "show_pixel_descriptions" so I did choose the shorter "show_pixel_formats". W

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_pixel_formats option (was: add -show_pixel_descriptions option)

2014-09-11 Thread Stefano Sabatini
On date Thursday 2014-09-11 09:54:42 +0200, Tobias Rapp encoded: > Have now added documentation and cleaned up some unused variables > with the new patch. Also on a second thought the option names > "show_pixel_format_descriptions" or "show_pixel_formats" seem less > confusing to me than "show_pixe

Re: [FFmpeg-devel] [PATCH] lavf/segment: add segment_format_options option

2014-09-11 Thread Stefano Sabatini
On date Wednesday 2014-09-10 17:57:04 +0800, Steven Liu encoded: > What about push this option into format hls? Please open a ticket on trac so I won't forget it (or alternatively provide a patch). -- FFmpeg = Fiendish & Furious Mournful Perennial Extroverse Gospel ___

Re: [FFmpeg-devel] [PATCH] doc/examples/transcoding: use av_packet_rescale_ts()

2014-09-11 Thread Michael Niedermayer
On Wed, Sep 10, 2014 at 12:17:11PM +0200, Stefano Sabatini wrote: > Simplify. > --- > doc/examples/transcoding.c | 36 +--- > 1 file changed, 9 insertions(+), 27 deletions(-) should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040

[FFmpeg-devel] [PATCH] lavf/hlsenc: add hls_ts_options option

2014-09-11 Thread Steven Liu
Signed-off-by: Steven Liu Signed-off-by: Stefano Sabatini --- doc/muxers.texi | 5 + libavformat/hlsenc.c | 24 ++-- 2 files changed, 27 insertions(+), 2 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 57e81f4..cc62705dc 100644 --- a/doc/muxers.t

Re: [FFmpeg-devel] [PATCH] avformat/rmdec: very basic MLTI support

2014-09-11 Thread Michael Niedermayer
On Tue, Sep 09, 2014 at 01:09:49AM +0200, Michael Niedermayer wrote: > Fixes Ticket2152 > > Signed-off-by: Michael Niedermayer > --- > libavformat/rmdec.c | 15 +++ > 1 file changed, 15 insertions(+) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78

[FFmpeg-devel] [PATCH 1/1] configure: fix check_cmd in check_host_cpp

2014-09-11 Thread Jörg Krause
Use correct cpp and c flags variables for the host libc. Signed-off-by: Jörg Krause --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index e66519d..7a711a2 100755 --- a/configure +++ b/configure @@ -1244,7 +1244,7 @@ check_host_cpp(){ l

Re: [FFmpeg-devel] [PATCH] lavf/segment: add segment_format_options option

2014-09-11 Thread Steven Liu
Okay, I'vemake a patch for the hls with this option :-) 2014-09-11 18:13 GMT+08:00 Stefano Sabatini : > On date Wednesday 2014-09-10 17:57:04 +0800, Steven Liu encoded: > > What about push this option into format hls? > > Please open a ticket on trac so I won't forget it (or alternatively >

Re: [FFmpeg-devel] [PATCH 1/1] configure: fix check_cmd in check_host_cpp

2014-09-11 Thread Benoit Fouet
Hi, - Mail original - > Use correct cpp and c flags variables for the host libc. > > Signed-off-by: Jörg Krause > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index e66519d..7a711a2 100755 > --- a/configure > +++ b/con

[FFmpeg-devel] [PATCH] img2dec: make probing score zero for known file extensions.

2014-09-11 Thread Benoit Fouet
When a known extension has its own probing function, make img_read_probe return 0, so that the probing functions are actually called instead of relying only on the file extensions. Fixes issue #3901 --- libavformat/img2dec.c | 24 +++- 1 file changed, 23 insertions(+), 1 delet

Re: [FFmpeg-devel] [PATCH] lavf/hlsenc: add hls_ts_options option

2014-09-11 Thread Stefano Sabatini
On date Thursday 2014-09-11 19:52:07 +0800, Steven Liu encoded: > > Signed-off-by: Steven Liu > Signed-off-by: Stefano Sabatini Probably you should remove this, and mention the equivalent commit in my patch (referencing the hash is fine). > --- > doc/muxers.texi | 5 + > libavforma

[FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Grace Ryan
Hi, I present this patch, which is to fix the bug for the calculation of 'ff_vector_fmul_add_altivec' when run '/libavutil/float_altivec-test' on POWER8 little endian. The fate test result can be found on http://fate.ffmpeg.org/ by search "ibmcrl", also attached here to facilitate the review:

[FFmpeg-devel] [PATCH 2/4] avformat/img2dec: reduce bmppipe probe score

2014-09-11 Thread Michael Niedermayer
bmp pipe needs the bmp parser which is not bug free and should thus not be favored over the bmp image2 demuxer Signed-off-by: Michael Niedermayer --- libavformat/img2dec.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index

[FFmpeg-devel] [PATCH 1/4] avformat/utils: return EINVAL from seek_frame_generic() when the requested target appears to be outside the file duration

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/utils.c |4 +-- tests/ref/seek/acodec-adpcm-swf |8 ++--- tests/ref/seek/acodec-adpcm-swf-trellis |8 ++--- tests/ref/seek/acodec-mp2|8 ++--- tests/ref/seek/lavf-avi

[FFmpeg-devel] [PATCH 4/4] avformat/img2dec: Fail probing when no data is yet available and the filename contains no number/glob patterns either.

2014-09-11 Thread Michael Niedermayer
Fixes Ticket3901 Signed-off-by: Michael Niedermayer --- libavformat/img2dec.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 9f48c5d..a21429f 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -162,6 +162,8 @@ static in

[FFmpeg-devel] [PATCH 3/4] avformat/format: Run image2 probe again when file content data is available

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/format.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/format.c b/libavformat/format.c index 828ab52..006cc87 100644 --- a/libavformat/format.c +++ b/libavformat/format.c @@ -202,7 +202,7 @@ AVInputFormat *av_

Re: [FFmpeg-devel] [PATCH] img2dec: make probing score zero for known file extensions.

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 03:48:21PM +0200, Benoit Fouet wrote: > When a known extension has its own probing function, make img_read_probe > return 0, so that the probing functions are actually called instead of > relying only on the file extensions. > > Fixes issue #3901 > --- > libavformat/img2de

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 10:18:00PM +0800, Grace Ryan wrote: > Hi, > > I present this patch, which is to fix the bug for the calculation of > 'ff_vector_fmul_add_altivec' when run '/libavutil/float_altivec-test' on > POWER8 little endian. > > The fate test result can be found on http://fate.ffmp

Re: [FFmpeg-devel] [PATCH 1/1] configure: fix check_cmd in check_host_cpp

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 02:48:15PM +0200, Benoit Fouet wrote: > Hi, > > - Mail original - > > Use correct cpp and c flags variables for the host libc. > > > > Signed-off-by: Jörg Krause > > --- > > configure | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 7:18 AM, Grace Ryan wrote: > Hi, > > I present this patch, which is to fix the bug for the calculation of > 'ff_vector_fmul_add_altivec' when run '/libavutil/float_altivec-test' on > POWER8 little endian. > > The fate test result can be found on http://fate.ffmpeg.org/ by

Re: [FFmpeg-devel] [PATCH 1/4] avformat/utils: return EINVAL from seek_frame_generic() when the requested target appears to be outside the file duration

2014-09-11 Thread Reimar Döffinger
On Thu, Sep 11, 2014 at 04:48:00PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer Isn't that potentially a bit confusing? EINVAL sounds to me like I would expect a programming error, inputting something I should have known to be invalid. That's not quite the case for out-o

Re: [FFmpeg-devel] [PATCH 1/4] avformat/utils: return EINVAL from seek_frame_generic() when the requested target appears to be outside the file duration

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 08:08:43PM +0200, Reimar Döffinger wrote: > On Thu, Sep 11, 2014 at 04:48:00PM +0200, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > Isn't that potentially a bit confusing? > EINVAL sounds to me like I would expect a programming error, > inputting so

[FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 4fa1174..44273cb 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -80,7 +80,9 @@ static const int ass_libav

[FFmpeg-devel] [PATCH 2/3] avfilter/ass: better log level mapping

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 44273cb..c9c21a7 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -68,14 +68,14 @@ typedef st

[FFmpeg-devel] [PATCH 3/3] avfilter/ass: add shaping option

2014-09-11 Thread Clément Bœsch
The documentation is mostly based on ass.h public header. --- I needed this for https://github.com/libass/libass/issues/135 TODO: bump lavfi micro --- doc/filters.texi | 20 libavfilter/vf_subtitles.c | 7 +++ 2 files changed, 27 insertions(+) diff --git a/do

Re: [FFmpeg-devel] suggested patch: avfilter/vf_subtitles: add support for subtitles font scaling

2014-09-11 Thread wm4
On Thu, 11 Sep 2014 03:58:07 +0400 Sergey wrote: > Hello. > > Recently I used ffmpeg to embed subtitles, and I needed to scale them. > > I thought "original_size" option scales subtitles, but it does not. > So I wrote a short patch for it to do that (attached) > > If that is considered too com

Re: [FFmpeg-devel] suggested patch: avfilter/vf_subtitles: add support for subtitles font scaling

2014-09-11 Thread Clément Bœsch
On Thu, Sep 11, 2014 at 09:21:03PM +0200, wm4 wrote: > On Thu, 11 Sep 2014 03:58:07 +0400 > Sergey wrote: > > > Hello. > > > > Recently I used ffmpeg to embed subtitles, and I needed to scale them. > > > > I thought "original_size" option scales subtitles, but it does not. > > So I wrote a shor

[FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
Replace with FFmpeg git-howto. Signed-off-by: Lou Logan --- src/contact | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/contact b/src/contact index 00cb44f..96f7378 100644 --- a/src/contact +++ b/src/contact @@ -26,11 +26,8 @@ text/plain or even better t

[FFmpeg-devel] FreeSwitch Video decoding (consulting opportunity)

2014-09-11 Thread Luciano Panaro
Hi everyone, In the company I work at we are trying to build a decoder for a video format generated by Freeswitch [1], a SIP platform through which we are recording a+v. We are using a module from FS called mod_fsv (details below). We are able to decode audio successfully but video is not being g

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
Lou Logan lrcd.com> writes: > - http://www.mplayerhq.hu/DOCS/tech/patches.txt";> > MPlayer patch guidelines I understand that this is not ideal but ... > + Using git to develop FFmpeg ... while this contains a lot of useful information it is missing (nearly) all relevant points from the exi

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > > +t1 = vec_perm(d, edges, vcprm(s0,s1,s2,s3)); > > +t0 = vec_perm(edges, d, vcprm(s0,s1,s2,s3)); > > ...you are using the old variable names, which I am > pretty sure won't compile. I cannot explain why but it compiles and passes fate here on L

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 1:57 PM, Carl Eugen Hoyos wrote: > Lou Logan lrcd.com> writes: > >> - http://www.mplayerhq.hu/DOCS/tech/patches.txt";> >> MPlayer patch guidelines > > I understand that this is not ideal but ... > >> + Using git to develop FFmpeg > > ... while this contains a lot of useful

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > Do you volunteer to port the relevant points to > developer.texi or git-howto.texi? No, I am currently testing ppc;-) Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 1:59 PM, Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > >> > +t1 = vec_perm(d, edges, vcprm(s0,s1,s2,s3)); >> > +t0 = vec_perm(edges, d, vcprm(s0,s1,s2,s3)); >> >> ...you are using the old variable names, which I am >> pretty sure won't compile.

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread wm4
On Thu, 11 Sep 2014 11:37:44 -0800 Lou Logan wrote: > Replace with FFmpeg git-howto. > > Signed-off-by: Lou Logan > --- > src/contact | 7 ++- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/src/contact b/src/contact > index 00cb44f..96f7378 100644 > --- a/src/contact >

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:00, Timothy Gu wrote: > On Thu, Sep 11, 2014 at 1:57 PM, Carl Eugen Hoyos wrote: >> Lou Logan lrcd.com> writes: >> >>> - http://www.mplayerhq.hu/DOCS/tech/patches.txt";> >>> MPlayer patch guidelines >> >> I understand that this is not ideal but ... >> >>> + Using git to develop F

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
On Thu, 11 Sep 2014 20:57:27 + (UTC), Carl Eugen Hoyos wrote: > Lou Logan lrcd.com> writes: > > > - http://www.mplayerhq.hu/DOCS/tech/patches.txt";> > > MPlayer patch guidelines > > I understand that this is not ideal but ... > > > + Using git to develop FFmpeg > > ... while this contain

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote: > --- > libavfilter/vf_subtitles.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c > index 4fa1174..44273cb 100644 > --- a/libavfilter/vf_subtitles.c

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 11:53:57PM +0200, Michael Niedermayer wrote: > On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote: [...] > btw, in > [2] = AV_LOG_WARNING, /* MSGL_WARN */ > [3] = AV_LOG_WARNING, /* */ > [4] = AV_LOG_INFO, /* MSGL_INFO */ > [5] = AV_LOG

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:10, James Darnley wrote: > I support the change and I am willing to look at adding and adapting > some relevant points from the mplayer text. Point 0, "diffs please", is covered by our request for git format-patch in the patch submission checklist[1] point 2 and the git how-to pag

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
James Darnley gmail.com> writes: > Point 2, "unified diffs", is covered by out request > for patches. Are we doing to allow git diff dumps? While I always send unified diffs, some people (still) send other diffs, and one regular contributor sends diffs in his own format, so this point is impo

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
Lou Logan lrcd.com> writes: > Anyway, we also have the quite verbose > http://ffmpeg.org/developer.html But you did not replace the MPlayer link by this link but by one that explains git usage which I consider important but different. Carl Eugen _

[FFmpeg-devel] [PATCH] x86/motion_est: port mmxext and sse2 sad functions to yasm

2014-09-11 Thread James Almer
Also add sse2 versions of sad16_x2, sad16_y2 and sad16_xy2, and a missing c->pix_abs[0][0] initialization. Signed-off-by: James Almer --- Not benched. TODO: Port mmx. libavcodec/x86/me_cmp.asm| 229 +++ libavcodec/x86/me_cmp_init.c | 194 +---

[FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Carl Eugen Hoyos
Hi! Attached patch fixes fate-webm-dash-manifest on aix. I cannot reproduce the problem on other systems and I don't know if the patch is correct. Please comment, Carl Eugen diff --git a/libavformat/webmdashenc.c b/libavformat/webmdashenc.c index 77f6170..849b241 100644 --- a/libavformat/webmdas

[FFmpeg-devel] [PATCH 2/6] docs: add example around the suggested commit message format

2014-09-11 Thread James Darnley
It gets rendered inline in HTML and becomes not very clear. --- doc/developer.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index e51e97a..f1fed5b 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -323,9 +323,12 @@ Always fill out the

[FFmpeg-devel] [PATCH 0/6] mplayer pacthes text missing points

2014-09-11 Thread James Darnley
This adds some text covering points from that mplayer text which are missing in our docs. I welcome your comments, particularly on the language used. The first two patches are really just cosmetic changes but I noticed them while reading the file. James Darnley (6): docs: add missing code form

[FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-11 Thread James Darnley
--- doc/developer.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index 1e1d3b8..e51e97a 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -328,7 +328,7 @@ area changed: Short 1 line description details describing wh

[FFmpeg-devel] [PATCH 3/6] docs: mention the preferred character to use in options

2014-09-11 Thread James Darnley
This addresses point 12 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/d

[FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread James Darnley
This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git a/doc

[FFmpeg-devel] [PATCH 6/6] docs: mention when and how one should be added as an author

2014-09-11 Thread James Darnley
This addresses point 13 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git a/doc

Re: [FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > --- > doc/developer.texi | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) I would use @command{}. [...] Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.o

[FFmpeg-devel] [PATCH 4/6] docs: mention the comment policy

2014-09-11 Thread James Darnley
This addresses point 6 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/do

Re: [FFmpeg-devel] [PATCH 2/6] docs: add example around the suggested commit message format

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > It gets rendered inline in HTML and becomes not very clear. > --- > doc/developer.texi | 3 +++ > 1 file changed, 3 insertions(+) OK. [...] Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffm

Re: [FFmpeg-devel] [PATCH 3/6] docs: mention the preferred character to use in options

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > This addresses point 12 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/developer.

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > This addresses point 11 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/developer.

Re: [FFmpeg-devel] [PATCH 4/6] docs: mention the comment policy

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > This addresses point 6 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/developer.t

Re: [FFmpeg-devel] [PATCH 6/6] docs: mention when and how one should be added as an author

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley wrote: > This addresses point 13 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/developer.

Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > Attached patch fixes fate-webm-dash-manifest on aix. This is the failure: $ gmake SAMPLES=fate-suite/ fate-webm-dash-manifest TESTwebm-dash-manifest diff: Missing newline at the end of file tests/data/fate/webm-dash-manifest. --- ./tests/ref/fate/webm-da

[FFmpeg-devel] [PATCH v2] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
Replace with FFmpeg Developer Documentation and mention git-howto. Signed-off-by: Lou Logan --- src/contact | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/contact b/src/contact index 00cb44f..1043ec9 100644 --- a/src/contact +++ b/src/contact @@ -26,11 +26,10 @@

[FFmpeg-devel] [PATCH 1/2] avformat/network: add union for avoiding strict aliassing violations with sockaddr*

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/network.h |8 1 file changed, 8 insertions(+) diff --git a/libavformat/network.h b/libavformat/network.h index d89a62d..d4e1a89 100644 --- a/libavformat/network.h +++ b/libavformat/network.h @@ -72,6 +72,14 @@ int ff_neterrno(void);

[FFmpeg-devel] [PATCH 2/2] avformat/rtpproto: fix strict aliasing violations with sockaddr

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/rtpproto.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c index a8cbd97..62f77da 100644 --- a/libavformat/rtpproto.c +++ b/libavformat/rtpproto.c @@ -130,23 +130,2

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread Michael Niedermayer
On Fri, Sep 12, 2014 at 01:31:29AM +0200, James Darnley wrote: > This addresses point 11 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/de

Re: [FFmpeg-devel] [PATCH 2/6] docs: add example around the suggested commit message format

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 04:33:39PM -0700, Timothy Gu wrote: > On Thu, Sep 11, 2014 at 4:31 PM, James Darnley > wrote: > > It gets rendered inline in HTML and becomes not very clear. > > --- > > doc/developer.texi | 3 +++ > > 1 file changed, 3 insertions(+) > > OK. applied thanks [...] -- M

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread wm4
On Fri, 12 Sep 2014 01:31:29 +0200 James Darnley wrote: > This addresses point 11 from the mplayer patches text file. Discussion about > removing a link to this document can be found in the thread on ffmpeg-devel > titled: [PATCH] web/contact: replace MPlayer guidelines link > --- > doc/develop

[FFmpeg-devel] [PATCH] add hls_ts_option into hlsenc

2014-09-11 Thread Steven Liu
same as COMMITID 4f5493fe2380ad4aba67759baa7d7d4437f2e776 --- doc/muxers.texi | 5 + libavformat/hlsenc.c | 24 +++- 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 57e81f4..cc62705dc 100644 --- a/doc/muxers.texi

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_pixel_formats option

2014-09-11 Thread Tobias Rapp
On 11.09.2014 12:12, Stefano Sabatini wrote: On date Thursday 2014-09-11 09:54:42 +0200, Tobias Rapp encoded: Have now added documentation and cleaned up some unused variables with the new patch. Also on a second thought the option names "show_pixel_format_descriptions" or "show_pixel_formats" s

Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Nicolas George
Le sextidi 26 fructidor, an CCXXII, Carl Eugen Hoyos a écrit : > Attached patch fixes fate-webm-dash-manifest on aix. > I cannot reproduce the problem on other systems and I > don't know if the patch is correct. I do not know that protocol, but adding a final newline at the end of an XML file see