Re: [FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread James Almer
On 19/09/14 8:24 PM, Michael Niedermayer wrote: > On Fri, Sep 19, 2014 at 07:18:10PM -0300, James Almer wrote: >> Also add mmxext versions of vsad8 and vsad_intra8, and sse2 versions of >> vsad16 and vsad_intra16. >> Since vsad8 and vsad16 are not bitexact, they are accordingly marked as >> approxi

Re: [FFmpeg-devel] ffprobe shows errors on ts file

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 08:23:34PM +0200, airwolf wrote: > Hi, > > following file shows a bunch of errors in ffprobe: > https://www.dropbox.com/s/04djuv4zn0l4lkx/espn.ts?dl=0 > > I'm not an expert in decoding that's why I try to ask here. > I converted the file to mp4 and used the mp4toannexb fil

Re: [FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 07:18:10PM -0300, James Almer wrote: > Also add mmxext versions of vsad8 and vsad_intra8, and sse2 versions of > vsad16 and vsad_intra16. > Since vsad8 and vsad16 are not bitexact, they are accordingly marked as > approximate. > > Signed-off-by: James Almer > --- > libavc

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 10:57:41AM -0700, Nicholas Robbins wrote: > > On Friday, September 19, 2014 12:31 PM, Carl Eugen Hoyos > > wrote: > > > > > > > Nicholas Robbins ffmpeg.org> writes: > > > >> +{ "only_dec_forced", "Only decode forced > > subtitles", > > > > Please use the same

[FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread James Almer
Also add mmxext versions of vsad8 and vsad_intra8, and sse2 versions of vsad16 and vsad_intra16. Since vsad8 and vsad16 are not bitexact, they are accordingly marked as approximate. Signed-off-by: James Almer --- libavcodec/x86/me_cmp.asm| 161 +++ lib

Re: [FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread James Almer
On 19/09/14 6:41 PM, Michael Niedermayer wrote: >> +mova m1, [pb_127] > isnt this supposed to be 128 ? Yes, will send a fixed version in a moment. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 05:04:49PM -0300, James Almer wrote: > Also add mmxext versions of vsad8 and vsad_intra8, and sse2 versions of > vsad16 and vsad_intra16. > Since vsad8 and vsad16 are not bitexact, they are accordingly marked as > approximate. > > Signed-off-by: James Almer > --- > Unrelat

Re: [FFmpeg-devel] [PATCH] Add _DEFAULT_SOURCE to hide warning about _SVID_SOURCE depreciation

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 02:56:00PM -0500, Daniel Bomar wrote: > As suggested, this just adds _DEFAULT_SOURCE while preserving > _SVID_SOURCE for whatever old systems still need it. > > Signed-off-by: Daniel Bomar > --- > libavformat/os_support.c | 1 + > 1 file changed, 1 insertion(+) applied

[FFmpeg-devel] [PATCH] x86/me_cmp: port mmxext vsad functions to yasm

2014-09-19 Thread James Almer
Also add mmxext versions of vsad8 and vsad_intra8, and sse2 versions of vsad16 and vsad_intra16. Since vsad8 and vsad16 are not bitexact, they are accordingly marked as approximate. Signed-off-by: James Almer --- Unrelated to this patch, but this simd implementation of vsad is *far* from approxi

[FFmpeg-devel] [PATCH] Add _DEFAULT_SOURCE to hide warning about _SVID_SOURCE depreciation

2014-09-19 Thread Daniel Bomar
As suggested, this just adds _DEFAULT_SOURCE while preserving _SVID_SOURCE for whatever old systems still need it. Signed-off-by: Daniel Bomar --- libavformat/os_support.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/os_support.c b/libavformat/os_support.c index e8f063a..f9d6e

[FFmpeg-devel] [PATCH] Only decode forced subtitle entries from dvdsub track.

2014-09-19 Thread Nicholas Robbins
Signed-off-by: Nicholas Robbins --- libavcodec/dvdsubdec.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/dvdsubdec.c b/libavcodec/dvdsubdec.c index 7355c03..de1ecbf 100644 --- a/libavcodec/dvdsubdec.c +++ b/libavcodec/dvdsubdec.c @@ -39,6 +39,7 @@ typedef struct DVDSubContex

Re: [FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-19 Thread Clément Bœsch
On Fri, Sep 19, 2014 at 09:00:28PM +0200, wm4 wrote: > On Fri, 19 Sep 2014 20:49:49 +0200 > Clément Bœsch wrote: > > > After this the order from the original file is stored through readorder > > when doing ffmpeg -i input.ass -c copy output.mkv. > > > > And now that the ASS muxer honors the Read

Re: [FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-19 Thread wm4
On Fri, 19 Sep 2014 20:49:49 +0200 Clément Bœsch wrote: > After this the order from the original file is stored through readorder > when doing ffmpeg -i input.ass -c copy output.mkv. > > And now that the ASS muxer honors the ReadOrder, extracting the ass back > (without transcoding) restores the

[FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-19 Thread Clément Bœsch
After this the order from the original file is stored through readorder when doing ffmpeg -i input.ass -c copy output.mkv. And now that the ASS muxer honors the ReadOrder, extracting the ass back (without transcoding) restores the original order. TODO: micro bump --- libavformat/assdec.c | 25 ++

[FFmpeg-devel] ffprobe shows errors on ts file

2014-09-19 Thread airwolf
Hi, following file shows a bunch of errors in ffprobe: https://www.dropbox.com/s/04djuv4zn0l4lkx/espn.ts?dl=0 I'm not an expert in decoding that's why I try to ask here. I converted the file to mp4 and used the mp4toannexb filter, then I used h264analyze. You can find the output below: https:/

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
> On Friday, September 19, 2014 1:15 PM, wm4 wrote: > > Sorry, but didn't the discussion go towards that ffmpeg.c should filter > out subtitles flagged as forced? > > And in fact, the PGS decoder should be changed to flag forced subs > correctly. I've never peeked inside ffmpeg.c so I don't

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
> On Friday, September 19, 2014 12:31 PM, Carl Eugen Hoyos > wrote: > > Nicholas Robbins ffmpeg.org> writes: > >> +{ "only_dec_forced", "Only decode forced > subtitles", > > Please use the same option name as for pgs subtitles. > > Carl Eugen Ok, I'll do that. Also I noticed that

Re: [FFmpeg-devel] [PATCH] Change deprecated use of _SVID_SOURCE to _DEFAULT_SOURCE

2014-09-19 Thread wm4
On Fri, 19 Sep 2014 08:19:34 -0500 Daniel Bomar wrote: > What should be tested in configure? IMO take this patch and and if it > breaks some old systems (not sure why it would) have a configure > option to revert this on those systems. I suppose it should test inet_aton, which is apparently why

Re: [FFmpeg-devel] [PATCH] avcodec/dv_profile: deprecate internal function that shouldn't be public

2014-09-19 Thread James Almer
On 19/09/14 6:58 AM, Michael Niedermayer wrote: > On Fri, Sep 19, 2014 at 01:50:28AM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> Unlike us, libav got rid of the internal symbols before the header became >> public on >> an actual relase, so we're stuck with it until next bum

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread wm4
On Fri, 19 Sep 2014 07:33:26 -0700 Nicholas Robbins wrote: > Ok, not to beat a dead horse, but I think I've gotten this working. I think > it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't > produce mangled tracks. Patch at end of email. (Sorry for the inline patch) > If

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 07:33:26AM -0700, Nicholas Robbins wrote: > Ok, not to beat a dead horse, but I think I've gotten this working. I think > it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't > produce mangled tracks. Patch at end of email. (Sorry for the inline patch)

Re: [FFmpeg-devel] [PATCH] Change deprecated use of _SVID_SOURCE to _DEFAULT_SOURCE

2014-09-19 Thread Carl Eugen Hoyos
Daniel Bomar gmail.com> writes: > What should be tested in configure? You wrote that compilation fails for you with an error, I originally assumed this can be reproduced with current FFmpeg git head (rereading our mail, I am not so sure). If compilation fails, this can be tested in configure

Re: [FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Carl Eugen Hoyos
Nicholas Robbins ffmpeg.org> writes: > +{ "only_dec_forced", "Only decode forced subtitles", Please use the same option name as for pgs subtitles. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo

Re: [FFmpeg-devel] [PATCH] Add input support for DeckLink devices.

2014-09-19 Thread Timothy Gu
On Fri, Sep 19, 2014 at 8:53 AM, Amnon Israely wrote: >> >> >> What do you think might be the problem here? >> >> > I changed streams order in deckling_dec.cpp > and Video now on 0:0 Video is not guaranteed to be always on 0:0, and therefore you are trying to fix a nonexistent problem here. [...

Re: [FFmpeg-devel] [PATCH] Add input support for DeckLink devices.

2014-09-19 Thread Amnon Israely
> > > What do you think might be the problem here? > > I changed streams order in deckling_dec.cpp and Video now on 0:0 /* Setup streams. */ st = avformat_new_stream(avctx, NULL); if (!st) { av_log(avctx, AV_LOG_ERROR, "Cannot add stream\n"); goto error; } st

[FFmpeg-devel] [patch] Only decode forced dvd-subtitles

2014-09-19 Thread Nicholas Robbins
Ok, not to beat a dead horse, but I think I've gotten this working. I think it might work better in lavc/dvdsubdec.c. AFAICT this works and doesn't produce mangled tracks. Patch at end of email. (Sorry for the inline patch) If this is a good patch, I'll add the appropriate documentation and do a

Re: [FFmpeg-devel] [PATCH] postproc: add basic deblock filter visualization support

2014-09-19 Thread Clément Bœsch
On Fri, Sep 19, 2014 at 03:44:31PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libpostproc/postprocess.c | 36 > +++- > libpostproc/postprocess_internal.h |1 + > libpostproc/postprocess_template.c | 14 +

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 08:08:10AM +0200, Reimar Döffinger wrote: > On 19 September 2014 07:34:11 CEST, Pascal Massimino > wrote: > >Hi Reimar, > >while i don't necessarily disagree with the above generally speaking > >[*], > >let me repeat that > >this is a pragmatic choice. I'm not going to giv

[FFmpeg-devel] [PATCH] postproc: add basic deblock filter visualization support

2014-09-19 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libpostproc/postprocess.c | 36 +++- libpostproc/postprocess_internal.h |1 + libpostproc/postprocess_template.c | 14 ++ 3 files changed, 46 insertions(+), 5 deletions(-) diff --git a/libpostproc

Re: [FFmpeg-devel] [PATCH] Change deprecated use of _SVID_SOURCE to _DEFAULT_SOURCE

2014-09-19 Thread Daniel Bomar
What should be tested in configure? IMO take this patch and and if it breaks some old systems (not sure why it would) have a configure option to revert this on those systems. On Thu, Sep 18, 2014 at 5:39 PM, Carl Eugen Hoyos wrote: > Daniel Bomar gmail.com> writes: > >> This is from /usr/includ

Re: [FFmpeg-devel] UDP TS feed switching from EAC3 to AC3 audio

2014-09-19 Thread Michael Niedermayer
On Thu, Sep 18, 2014 at 02:35:42PM -0600, Ryan Owen wrote: > I have a TS feed that I'm reading with libavformat/libavcodec over > multicast. The feed comes from an over-the-air channel, where the video is > mpeg2 and the audio is AC3. > > Occasionally there are some dropped packets, and that some

Re: [FFmpeg-devel] [PATCH]: libavcodec/webp

2014-09-19 Thread Michael Niedermayer
On Thu, Sep 18, 2014 at 10:34:11PM -0700, Pascal Massimino wrote: > Hi Reimar, > > On Thu, Sep 18, 2014 at 12:28 PM, Reimar Döffinger > wrote: > > > On 18 September 2014 10:55:00 CEST, Pascal Massimino < > > pascal.massim...@gmail.com> wrote: > > >Hi Reimar, > > > > > >On Thu, Sep 18, 2014 at 9:

Re: [FFmpeg-devel] More specific error codes

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 12:59:35PM +0300, Andrey Utkin wrote: > Thank you for your feedback. > Any more comments/advices? > Are patch submissions on this topic welcome? yes, but please also seperate addition of codes and their use so you dont waste time changing alot of code to a error code that

Re: [FFmpeg-devel] Questions on adding a FATE test for the HLS muxer

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 04:15:13AM +, Raento Mika wrote: > On 19/09/14 01:27, "Michael Niedermayer" wrote: > > >On Thu, Sep 18, 2014 at 06:05:38PM +, Raento Mika wrote: > >> Hiy'all > >> > >> As suggested by Michael, I'm trying to add a FATE test for the new > >> single_file option to th

Re: [FFmpeg-devel] [PATCH v2 1/1] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-19 Thread Hendrik Leppkes
On Thu, Sep 18, 2014 at 8:24 PM, Jörg Krause wrote: > > On 09/18/2014 07:10 PM, Reimar Döffinger wrote: > >> On 18.09.2014, at 17:26, Michael Niedermayer wrote: >> >>> On Thu, Sep 18, 2014 at 04:22:27PM +0200, Jörg Krause wrote: >>> glibc, uclibc, and musl uses feature test macros to expose

Re: [FFmpeg-devel] More specific error codes

2014-09-19 Thread Andrey Utkin
Thank you for your feedback. Any more comments/advices? Are patch submissions on this topic welcome? -- Andrey Utkin ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/dv_profile: deprecate internal function that shouldn't be public

2014-09-19 Thread Michael Niedermayer
On Fri, Sep 19, 2014 at 01:50:28AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > Unlike us, libav got rid of the internal symbols before the header became > public on > an actual relase, so we're stuck with it until next bump. > I renamed the function to ff_ since it's only used

Re: [FFmpeg-devel] [PATCH] support for latest git of libilbc

2014-09-19 Thread Michael Niedermayer
On Thu, Sep 18, 2014 at 09:40:20PM -0700, Timothy Gu wrote: > On Thu, Sep 18, 2014 at 7:49 PM, Gianluigi Tiesi wrote: > > Hi, > > > > in the latest git commits of libilbc developers removed WebRtc_xxx > typedefs > > > > The attached patch uses int types instead, > > it's safe to apply also for pre