Re: [FFmpeg-devel] [PATCH] lavf: add samba protocol via libsmbclient

2014-07-13 Thread wm4
On Sat, 12 Jul 2014 18:20:46 +0200 Lukasz Marek lukasz.m.lu...@gmail.com wrote: --- Changelog | 1 + configure | 5 ++ doc/general.texi | 1 + doc/protocols.texi | 29 +++ libavformat/Makefile | 1 +

Re: [FFmpeg-devel] [PATCH] Revert lavc/utils: Do not require dimensions for PNG.

2014-07-14 Thread wm4
On Mon, 14 Jul 2014 01:57:52 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Jul 13, 2014 at 07:32:56PM +0100, Derek Buitenhuis wrote: mplayer-specifc hacks should not be in our codebase. mplayer should fix it's own code. It is not our responsibility to work around their broken

Re: [FFmpeg-devel] [PATCH] Revert lavc/utils: Do not require dimensions for PNG.

2014-07-14 Thread wm4
On Mon, 14 Jul 2014 09:15:20 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: I actually think it worked with a lot of formats, not just PNG (which is not an argument for the hack though). This is also related to encoders which can in principle support resolution changes. That is

Re: [FFmpeg-devel] [PATCH] libavu: add pkt_timebase to AVFrame.

2014-07-20 Thread wm4
On Sun, 20 Jul 2014 17:01:42 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Jul 20, 2014 at 04:26:01PM +0200, wm4 wrote: On Fri, 18 Jul 2014 13:12:39 +0200 Michael Niedermayer michae...@gmx.at wrote: On Fri, Jul 18, 2014 at 12:47:06PM +0200, Hendrik Leppkes wrote: Am

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-07-24 Thread wm4
On Thu, 24 Jul 2014 15:39:43 +0200 Nicolas George geo...@nsup.org wrote: Also deprecate adding a NULL frame to mark EOF. No, you can't do this. It breaks compatibility with Libav; also what's even the purpose of this (API) change? TODO APIchanges entry, version bump. Signed-off-by: Nicolas

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-27 Thread wm4
On Sun, 27 Jul 2014 23:56:21 +0200 Nicolas George geo...@nsup.org wrote: Le nonidi 9 thermidor, an CCXXII, wm4 a écrit : DVD images have metadata that tell you the number of subtitles in advance. Surely you are aware that ffmpeg is not capable to read it? I have the impression

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-29 Thread wm4
On Mon, 28 Jul 2014 17:21:34 +0200 Nicolas George geo...@nsup.org wrote: Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit : Ah! Thank you very much for pointing me to the concat demuxer. I wasn't aware that it can be (ab)used to declare the presence of streams in an input file. I

Re: [FFmpeg-devel] rectification filter

2014-07-29 Thread wm4
On Tue, 29 Jul 2014 00:31:48 +0200 Daniel Oberhoff danieloberh...@gmail.com wrote: Hello, In the last days I effectively rewrote the rectification filter found in the frei0r suite as a libavfilter video filter and would like to submit this to ffmpeg. It probably needs some cleanup, which

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-07-29 Thread wm4
On Thu, 24 Jul 2014 15:49:33 +0200 Nicolas George geo...@nsup.org wrote: Le sextidi 6 thermidor, an CCXXII, wm4 a écrit : No, you can't do this. Yes, I can. It breaks compatibility with Libav; No, it does not. Removing it would break compatibility. Deprecating it does nothing

Re: [FFmpeg-devel] [PATCH 2/4] dict.c: Add av_dict_set_int helper function.

2014-07-30 Thread wm4
On Wed, 30 Jul 2014 20:38:06 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: This allows getting rid of the many, slightly differing, implementations of basically the same thing. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- doc/APIchanges | 3 +++

Re: [FFmpeg-devel] [PATCH 2/5] lavfi/buffersrc: add add av_buffersrc_close().

2014-08-04 Thread wm4
On Sun, 3 Aug 2014 15:15:37 +0200 Nicolas George geo...@nsup.org wrote: diff --git a/libavfilter/buffersrc.h b/libavfilter/buffersrc.h index ea34c04..28ca545 100644 --- a/libavfilter/buffersrc.h +++ b/libavfilter/buffersrc.h @@ -145,6 +145,8 @@ int av_buffersrc_add_frame(AVFilterContext

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread wm4
On Mon, 4 Aug 2014 20:47:37 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Mon, Aug 04, 2014 at 08:19:38PM +0200, Hendrik Leppkes wrote: Am 04.08.2014 19:59 schrieb Gisle Sælensminde gi...@snirklasjon.no: I'm trying to add support for closed captions in ffmpeg, namely cea608

Re: [FFmpeg-devel] [PATCH 1/2] avutil/libm: Replace macro based fminf() by function

2014-08-04 Thread wm4
On Mon, 04 Aug 2014 21:25:12 +0200 Bernd Kuhls bernd.ku...@t-online.de wrote: Michael Niedermayer michae...@gmx.at wrote in news:1402065736-24164-1-git- send-email-michae...@gmx.at: libavutil/libm.h |5 - Hi, the current code in libm.h breaks compilation when the libc does not

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-08-05 Thread wm4
On Tue, 22 Jul 2014 01:51:46 +0200 Michael Niedermayer michae...@gmx.at wrote: On Wed, Jul 16, 2014 at 05:06:27PM +0200, Clément Bœsch wrote: Also add an example exporting the MB information as a CSV stream. --- A bunch of questions stand: - Ideally the source for a given

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-08 Thread wm4
On Fri, 8 Aug 2014 15:06:20 +0100 Alessio Treglia ales...@debian.org wrote: Hi, On Fri, Aug 8, 2014 at 12:13 PM, Matthias Urlichs matth...@urlichs.de wrote: That leaves the question of the official opinion of the libav maintainers (pkg-multimedia-maintain...@lists.alioth.debian.org).

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-08 Thread wm4
On Sat, 9 Aug 2014 11:46:40 +1000 Jean-Yves Avenard jyaven...@gmail.com wrote: MythTV do not work against stock FFmpeg and never will. We run a heavily modified version of FFmpeg, you can call it a fork. In particular, we use our own mpeg-ts demuxer, we support things like MHEG,

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-08 Thread wm4
On Sat, 9 Aug 2014 01:28:05 -0400 compn te...@mi.rr.com wrote: On Mon, 04 Aug 2014 19:58:52 +0200 Gisle Sælensminde gi...@snirklasjon.no wrote: I'm trying to add support for closed captions in ffmpeg, namely cea608 and cea708. mythtv fork of ffmpeg has this support already. maybe we

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-10 Thread wm4
On Sat, 09 Aug 2014 20:25:09 +0200 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi Kieran, On 09.08.2014 19:26, Kieran Kunhya wrote: The reality is that in the current state of affairs static linking is the *only* way you are guaranteed to have the features you expect and

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-10 Thread wm4
On Sun, 10 Aug 2014 13:25:03 +0200 Nicolas George geo...@nsup.org wrote: Le tridi 23 thermidor, an CCXXII, wm4 a écrit : Unfortunately, FFmpeg vehemently resists against enabling lavr by default. If someone cared enough, they would be making their case: posting benchmarks, listing

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-10 Thread wm4
On Sun, 10 Aug 2014 09:43:03 +0200 Matthias Urlichs matth...@urlichs.de wrote: Hi, Andrew Kelley: It is unreasonable to expect no incompatible changes. When somebody renames constants, a compatibility #ifdef or two is not too much to ask, IMHO. AFAIK such a thing existed, but it's

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-11 Thread wm4
On Mon, 11 Aug 2014 21:17:18 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: Unfortunately this was not explicitly documented and thus might be very risky. But basically all uses I saw in FFmpeg had a memleak in these cases. It's the more convenient behavior, although on the other

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-11 Thread wm4
On Mon, 11 Aug 2014 20:40:28 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: Hello, Apologies for not being able to resist answering even if it is getting off-topic. On Sun, Aug 10, 2014 at 05:43:33PM -0400, Theodore Ts'o wrote: On Sun, Aug 10, 2014 at 12:25:33AM -0700, Andrew

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-12 Thread wm4
On Mon, 11 Aug 2014 18:34:23 -0400 Theodore Ts'o ty...@mit.edu wrote: On Mon, Aug 11, 2014 at 10:53:56PM +0200, wm4 wrote: To be fair, FFmpeg does its own manual symbol versioning by appending increasing numbers to function names. But the real problem are not these functions, but public

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-12 Thread wm4
On Tue, 12 Aug 2014 02:54:39 +0200 Matthias Urlichs matth...@urlichs.de wrote: Hi, wm4: Build something on a newer glibc system, and try to run the binary on an older system. It most likely won't work - even if it could in theory. (At least it was this way some years ago. Probably still

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-12 Thread wm4
On Tue, 12 Aug 2014 16:51:40 +0200 Matthias Urlichs matth...@urlichs.de wrote: Hi, wm4: In fact, the API cleanup is an ongoing process, and is what causes the incompatibilities with each release. For example, a C library should have a consistent naming schema. FFmpeg/Libav decided

Re: [FFmpeg-devel] [PATCH 1/7] doc/examples: remove unneeded NULL checks

2014-08-15 Thread wm4
On Sat, 16 Aug 2014 01:15:42 +0200 Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/examples/resampling_audio.c |3 +-- doc/examples/scaling_video.c|3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 1/7] doc/examples: remove unneeded NULL checks

2014-08-15 Thread wm4
On Sat, 16 Aug 2014 01:44:59 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sat, Aug 16, 2014 at 01:30:44AM +0200, wm4 wrote: On Sat, 16 Aug 2014 01:15:42 +0200 Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/examples

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread wm4
On Sat, 16 Aug 2014 23:29:56 +0800 Thomas Goirand z...@debian.org wrote: On 08/15/2014 11:53 PM, The Wanderer wrote: It's also something the Linux kernel is still doing, with apparent success. Yes, the Linux kernel is a successful project. Does this mean using a list for reviewing

Re: [FFmpeg-devel] [PATCH] avformat/metadata: Implement AVFMT_FLAG_NO_META_CONV

2014-08-16 Thread wm4
On Mon, 11 Aug 2014 15:15:45 +0300 Mohammad Alsaleh ce.mohammad.alsa...@gmail.com wrote: Decoders/parsers run ff_metadata_conv() unconditionally. This makes it impossible to retrieve or keep original metadata field names. If one wishes to skip ff_metadata_conv() calls at the

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-16 Thread wm4
On Sat, 16 Aug 2014 11:59:20 -0700 Russ Allbery r...@debian.org wrote: The problem, however, is that taking security seriously, while possibly necessary, is not sufficient. I'm glad that FFmpeg takes security seriously, but what FFmpeg needs is to *have fewer security bugs*. That is very

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-17 Thread wm4
On Sun, 17 Aug 2014 13:29:15 +0200 Nicolas George geo...@nsup.org wrote: Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit : When new cookie values (with the same name as an existing cookie) are returned in an HLS stream, the current implementation will append the new cookie to the

Re: [FFmpeg-devel] [RFC] removing libmpcodecs for reuniting purposes.

2014-08-17 Thread wm4
On Sun, 17 Aug 2014 19:04:35 -0400 compn te...@mi.rr.com wrote: libav brought up the point again that it doesnt like libmpcodecs. is there anyone using the remaining libmpcodecs filters ? most of the filters have been ported to libavfilter already. comments? +1000 Nobody cares about

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread wm4
On Sun, 24 Aug 2014 00:28:56 +0200 Clément Bœsch u...@pkh.me wrote: Hi, Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list between the two projects to start communicating again in sane terms. The proposition would be a mailing-list where the 2 projects would send the

Re: [FFmpeg-devel] Common mailing-list for API evolutions

2014-08-24 Thread wm4
On Sun, 24 Aug 2014 19:35:28 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Aug 24, 2014 at 02:54:19PM +0200, wm4 wrote: On Sun, 24 Aug 2014 00:28:56 +0200 Clément Bœsch u...@pkh.me wrote: Hi, Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list

Re: [FFmpeg-devel] [PATCH] Add SUP (raw BluRay PGS subtitle) muxer

2014-08-29 Thread wm4
On Fri, 29 Aug 2014 13:31:22 +0300 Petri Hintukainen phint...@gmail.com wrote: From: Petri Hintukainen phint...@users.sourceforge.net Fixes ticket #2208 --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supenc.c | 61

Re: [FFmpeg-devel] [PATCH] lavu/avstring: check for overlong encodings

2014-08-29 Thread wm4
On Thu, 28 Aug 2014 17:39:27 +0200 Stefano Sabatini stefa...@gmail.com wrote: Fix reopened trac ticket #1163. --- libavutil/avstring.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavutil/avstring.c b/libavutil/avstring.c index a63fb84..df27d5e

Re: [FFmpeg-devel] [PATCH] cabac: initialize all of ff_h264_cabac_tables programmatically.

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 18:41:52 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 18:29, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sat, Aug 30, 2014 at 06:20:56PM +0200, wm4 wrote: On Sat, 30 Aug 2014 18:18:41 +0200 Reimar Döffinger reimar.doeffin...@gmx.de

Re: [FFmpeg-devel] [PATCH] cabac: initialize all of ff_h264_cabac_tables programmatically.

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 19:58:11 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 19:46, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 19:01, wm4 nfx...@googlemail.com wrote: but still I wonder why you'd optimize for binary size, in exchange

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-30 Thread wm4
On Sat, 30 Aug 2014 18:16:27 -0400 Marcus Johnson bumblebritche...@gmail.com wrote: Here's a PGS subtitle sample from Eac3to: https://dl.dropboxusercontent.com/u/52358991/English.zip Thanks! The DTS fields in this one are all 0 too. ___

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
On Sun, 31 Aug 2014 10:24:13 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sun, Aug 31, 2014 at 10:15:56AM +0200, Hendrik Leppkes wrote: On Sun, Aug 31, 2014 at 9:41 AM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 30.08.2014, at 15:38, wm4 nfx...@googlemail.com wrote

[FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
--- I got the semantics of avio_read wrong: short reads on EOF are in fact allowed and don't return an error. Hopefully this is correct now... --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supdec.c | 92 3

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
On Sun, 31 Aug 2014 11:50:36 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sun, Aug 31, 2014 at 11:39:16AM +0200, wm4 wrote: I got the semantics of avio_read wrong: short reads on EOF are in fact allowed and don't return an error. Hopefully this is correct now... What I

[FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
--- Following the advice to use av_get_packet() etc. Tese functions still return partial packets, but mark them as corrupted - so returning them should be fine. --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supdec.c | 85

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
On Sun, 31 Aug 2014 14:25:21 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sun, Aug 31, 2014 at 01:04:29PM +0200, wm4 wrote: +static int sup_probe(AVProbeData *p) +{ +if (p-buf_size 2 || memcmp(p-buf, PG, 2)) +return 0; +return AVPROBE_SCORE_EXTENSION; I

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
On Sun, 31 Aug 2014 15:16:49 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sun, Aug 31, 2014 at 02:57:54PM +0200, wm4 wrote: On Sun, 31 Aug 2014 14:25:21 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sun, Aug 31, 2014 at 01:04:29PM +0200, wm4 wrote: +static

Re: [FFmpeg-devel] [PATCH] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
On Sun, 31 Aug 2014 15:06:08 + (UTC) Carl Eugen Hoyos ceho...@ag.or.at wrote: wm4 nfxjfg at googlemail.com writes: +return AVPROBE_SCORE_EXTENSION; I understand if you consider it not worth the effort, but ideally this would scan ahead several packets, and if it all

[FFmpeg-devel] [PATCH v4] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
--- Integrating Reimar's suggestions. No changes outside of the probe function. --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supdec.c | 107 +++ 3 files changed, 109 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-08-31 Thread wm4
--- Use AV_RB16 instead of memcpy. Don't use AVPROBE_SCORE_EXTENSION. --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/supdec.c | 107 +++ 3 files changed, 109 insertions(+) create mode 100644 libavformat/supdec.c

Re: [FFmpeg-devel] [PATCH v5] Add SUP/PGS subtitle demuxer

2014-09-01 Thread wm4
On Mon, 1 Sep 2014 08:34:52 + (UTC) Carl Eugen Hoyos ceho...@ag.or.at wrote: wm4 nfxjfg at googlemail.com writes: +avio_rb32(s-pb); /* discard DTS (usually 0, and useless) */ Would it be an option to set it dts to AV_NOPTS_VALUE if this value is 0 but use the value otherwise

Re: [FFmpeg-devel] ffmpeg and musl toolchain support

2014-09-01 Thread wm4
On Mon, 01 Sep 2014 16:14:52 +0200 Jörg Krause jkra...@posteo.de wrote: When building against musl instead of glibc, compilation fails at libavutil/error.c CC libavutil/error.o libavutil/error.c: In function 'av_strerror': libavutil/error.c:68:9: error: implicit declaration of function

Re: [FFmpeg-devel] [PATCH 1/4] libavutil/error: fix build with musl toolchain

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 12:33:26 +0200 Jörg Krause jkra...@posteo.de wrote: Add the feature test macro which is required for building with the musl toolchain. The feature test macro _XOPEN_SOURCE = 600 provides the XSI-compliant version of strerror_r(). Signed-off-by: Jörg Krause

[FFmpeg-devel] [PATCH 1/4] avformat/assdec: UTF-16 support

2014-09-02 Thread wm4
Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext. It also can work on a static buffer, needed for format probing. The FFTextReader wrapper now also takes care of skipping the UTF-8

[FFmpeg-devel] [PATCH 3/4] avformat/samidec: UTF-16 support

2014-09-02 Thread wm4
ff_smil_extract_next_chunk() is still used by RealText. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat/subtitles.h | 5 + 3 files changed, 27 insertions(+), 10 deletions(-) diff --git a/libavformat/samidec.c

Re: [FFmpeg-devel] [PATCH] ffv1enc: reduce stack usage.

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 20:32:57 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: A bit more complex than e.g. adding it to the context, but using the context for something that will be used only during initialization seemed a bit wasteful. Signed-off-by: Reimar Döffinger

Re: [FFmpeg-devel] [PATCH 1/4] avformat/assdec: UTF-16 support

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 21:05:08 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Tue, Sep 02, 2014 at 08:56:09PM +0200, wm4 wrote: Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around

Re: [FFmpeg-devel] [PATCH] ffv1enc: reduce stack usage.

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 21:18:24 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Tue, Sep 02, 2014 at 08:58:50PM +0200, wm4 wrote: On Tue, 2 Sep 2014 20:32:57 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: @@ -933,6 +938,7 @@ static av_cold int encode_init(AVCodecContext

Re: [FFmpeg-devel] [PATCH] ffv1enc: reduce stack usage.

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 21:54:36 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Tue, Sep 02, 2014 at 09:28:27PM +0200, wm4 wrote: I don't see anything wrong in the patch (well, maybe you should switch the code to the goto fail; idiom). I was just wondering whether

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-09-02 Thread wm4
On Fri, 29 Aug 2014 15:44:34 -0700 Peter Kasting pkast...@google.com wrote: From 1c94e78d2b2037d492ea5abb3edb7960c8e98a1d Mon Sep 17 00:00:00 2001 From: Peter Kasting pkast...@google.com Date: Fri, 29 Aug 2014 15:31:41 -0700 Subject: [PATCH] Fix MSVC warnings about possible value truncation.

Re: [FFmpeg-devel] [PATCH 1/4] avformat/assdec: UTF-16 support

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 23:18:02 +0200 Clément Bœsch u...@pkh.me wrote: On Tue, Sep 02, 2014 at 08:56:09PM +0200, wm4 wrote: Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext

Re: [FFmpeg-devel] [PATCH 2/4] avformat/srtdec: UTF-16 support

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 23:54:51 +0200 Clément Bœsch u...@pkh.me wrote: On Tue, Sep 02, 2014 at 08:56:10PM +0200, wm4 wrote: --- libavformat/srtdec.c| 22 +++-- libavformat/subtitles.c | 51 +++-- libavformat/subtitles.h | 33

Re: [FFmpeg-devel] [PATCH 3/4] avformat/samidec: UTF-16 support

2014-09-02 Thread wm4
On Wed, 3 Sep 2014 00:03:04 +0200 Clément Bœsch u...@pkh.me wrote: On Tue, Sep 02, 2014 at 08:56:11PM +0200, wm4 wrote: ff_smil_extract_next_chunk() is still used by RealText. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat

Re: [FFmpeg-devel] [PATCH] Fix MSVC warnings about possible value truncation.

2014-09-02 Thread wm4
On Tue, 2 Sep 2014 23:43:33 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Tue, Sep 02, 2014 at 01:13:27PM -0700, Peter Kasting wrote: On Sat, Aug 30, 2014 at 2:21 AM, wm4 nfx...@googlemail.com wrote: I'd expect it rather to hide bugs than to expose them. For example

Re: [FFmpeg-devel] [PATCH 2/4] avformat/srtdec: UTF-16 support

2014-09-03 Thread wm4
On Tue, 2 Sep 2014 23:54:51 +0200 Clément Bœsch u...@pkh.me wrote: On Tue, Sep 02, 2014 at 08:56:10PM +0200, wm4 wrote: --- libavformat/srtdec.c| 22 +++-- libavformat/subtitles.c | 51 +++-- libavformat/subtitles.h | 33

Re: [FFmpeg-devel] patch for x32 for libpostproc

2014-09-04 Thread wm4
On Wed, 3 Sep 2014 23:33:48 -0400 Reinhard Tartler siret...@gmail.com wrote: On Wed, Sep 3, 2014 at 9:34 PM, Michael Niedermayer michae...@gmx.at wrote: On Wed, Sep 03, 2014 at 08:22:43PM -0400, Reinhard Tartler wrote: On Wed, Sep 3, 2014 at 9:39 AM, Michael Niedermayer michae...@gmx.at

Re: [FFmpeg-devel] patch for x32 for libpostproc

2014-09-04 Thread wm4
On Thu, 4 Sep 2014 16:30:14 +0200 Michael Niedermayer michae...@gmx.at wrote: On Thu, Sep 04, 2014 at 03:53:55PM +0200, wm4 wrote: On Wed, 3 Sep 2014 23:33:48 -0400 Reinhard Tartler siret...@gmail.com wrote: On Wed, Sep 3, 2014 at 9:34 PM, Michael Niedermayer michae...@gmx.at wrote

Re: [FFmpeg-devel] [PATCH v2 1/4] libavutil/error: fix build with musl toolchain

2014-09-04 Thread wm4
On Thu, 04 Sep 2014 10:07:19 +0200 Jörg Krause jkra...@posteo.de wrote: On 09/03/2014 08:12 PM, Michael Niedermayer wrote: On Wed, Sep 03, 2014 at 11:53:39AM +0200, Jörg Krause wrote: Add the feature test macro which is required for building with the musl toolchain. The feature test

Re: [FFmpeg-devel] [PATCH v3] ffprobe: Add bits_per_component to stream output

2014-09-04 Thread wm4
On Thu, 04 Sep 2014 16:31:45 +0200 Tobias Rapp t.r...@noa-audio.com wrote: From 4a2290540ae0761c6ea2a5f12326ad8dd1257e57 Mon Sep 17 00:00:00 2001 From: Tobias Rapp t.r...@noa-audio.com Date: Thu, 4 Sep 2014 16:25:22 +0200 Subject: [PATCH] ffprobe: Add bits_per_component to stream output

Re: [FFmpeg-devel] [PATCH v2 1/4] libavutil/error: fix build with musl toolchain

2014-09-04 Thread wm4
On Thu, 4 Sep 2014 17:27:48 +0200 Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Sep 3, 2014 at 11:53 AM, Jörg Krause jkra...@posteo.de wrote: Add the feature test macro which is required for building with the musl toolchain. The feature test macro _XOPEN_SOURCE = 600 provides the

[FFmpeg-devel] [PATCH v2 4/6] avformat/realtextdec: UTF-16 support

2014-09-04 Thread wm4
Also remove ff_smil_extract_next_chunk - this was the last user of it. --- No changes. --- libavformat/realtextdec.c | 17 ++--- libavformat/subtitles.c | 9 - libavformat/subtitles.h | 5 - 3 files changed, 10 insertions(+), 21 deletions(-) diff --git

[FFmpeg-devel] [PATCH v2 3/6] avformat/samidec: UTF-16 support

2014-09-04 Thread wm4
ff_smil_extract_next_chunk() is still used by RealText. --- No changes. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat/subtitles.h | 5 + 3 files changed, 27 insertions(+), 10 deletions(-) diff --git a/libavformat/samidec.c

[FFmpeg-devel] [PATCH v2 1/6] avformat/assdec: UTF-16 support

2014-09-04 Thread wm4
Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext. It also can work on a static buffer, needed for format probing. The FFTextReader wrapper now also takes care of skipping the UTF-8

[FFmpeg-devel] [PATCH v2 2/6] avformat/srtdec: UTF-16 support

2014-09-04 Thread wm4
--- ff_subtitles_read_line() now returns an error if a 0 byte is encountered. This is faster and could be considered more correct. It also handles UTF recode errors better. --- libavformat/srtdec.c| 25 +--- libavformat/subtitles.c | 51

[FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-04 Thread wm4
--- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything. I didn't use ffio_ensure_seekback), because this apparently reallocates the buffer - which we certainly don't want, because the probe API doesn't

[FFmpeg-devel] [PATCH v2 5/6] avformat/srtdec: speed up probing

2014-09-04 Thread wm4
--- As suggested by ubitux. Note that the change to ff_subtitles_read_line() already cuts down the probing time a lot, so maybe this is not needed. --- libavformat/srtdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/srtdec.c b/libavformat/srtdec.c index b63d344..f5f3933

[FFmpeg-devel] [PATCH v3] avformat/srtdec: UTF-16 support

2014-09-05 Thread wm4
--- Fixes issues with ff_subtitles_read_line() EOF handling pointed out by ubitux on IRC. --- libavformat/srtdec.c| 25 +-- libavformat/subtitles.c | 53 +++-- libavformat/subtitles.h | 34 --- 3 files

Re: [FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-05 Thread wm4
On Sat, 6 Sep 2014 00:00:10 +0200 Michael Niedermayer michae...@gmx.at wrote: On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote: --- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-07 Thread wm4
On Sun, 7 Sep 2014 22:03:26 +0200 Clément Bœsch u...@pkh.me wrote: --- libavformat/matroska.c| 6 libavformat/matroskadec.c | 80 --- libavformat/matroskaenc.c | 79 -- libavformat/version.h

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 serg...@mail.ru 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

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 james.darn...@gmail.com 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

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add AVFMT_FILE to allow demuxers to be used with and without files

2014-09-12 Thread wm4
On Fri, 12 Sep 2014 12:29:36 +0200 Michael Niedermayer michae...@gmx.at wrote: Suggested-by: Benoit Fouet benoit.fo...@free.fr Signed-off-by: Michael Niedermayer michae...@gmx.at --- doc/APIchanges |3 +++ libavformat/avformat.h |8 libavformat/format.c |3

Re: [FFmpeg-devel] Should add AVProbeData change to API changes + release notes

2014-09-12 Thread wm4
On Fri, 12 Sep 2014 18:51:48 +0200 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 12.09.2014 15:18, Michael Niedermayer wrote: On Fri, Sep 12, 2014 at 01:54:36PM +0200, Andreas Cadhalpun wrote: On 11.08.2014 22:22, Michael Niedermayer wrote: On Mon, Aug 11, 2014 at

Re: [FFmpeg-devel] [PATCH] web/contact: clarify mailing list descriptions

2014-09-12 Thread wm4
On Fri, 12 Sep 2014 15:27:00 -0800 Lou Logan l...@lrcd.com wrote: Also slightly reorder mailing lists by category of development, user help, then logs. Also link some phrases to relevant pages. Signed-off-by: Lou Logan l...@lrcd.com --- src/contact | 25 - 1

Re: [FFmpeg-devel] [PATCH] doc/APIchanges: attempt to split releases

2014-09-14 Thread wm4
On Sun, 14 Sep 2014 08:56:02 +0200 Clément Bœsch u...@pkh.me wrote: Based on several git show origin/release/X.Y:doc/APIchanges. --- doc/APIchanges | 42 +- 1 file changed, 37 insertions(+), 5 deletions(-) diff --git a/doc/APIchanges

Re: [FFmpeg-devel] Branchpoint tags

2014-09-14 Thread wm4
On Sun, 14 Sep 2014 19:12:56 +0200 Michael Niedermayer michae...@gmx.at wrote: Hi Should we add git tags to the revissions where releases are branched off ? that way a git describe on master would look like: n2.5-dev-3-gb227be3 instead of: n2.0-11670-gb227be3 (or any other tag than

Re: [FFmpeg-devel] Branchpoint tags

2014-09-15 Thread wm4
On Sun, 14 Sep 2014 23:36:14 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep 14, 2014 at 07:31:42PM +0200, wm4 wrote: On Sun, 14 Sep 2014 19:12:56 +0200 Michael Niedermayer michae...@gmx.at wrote: Hi Should we add git tags to the revissions where releases

Re: [FFmpeg-devel] [WIP] lavfi: port mp/eq to a regular filter

2014-09-15 Thread wm4
On Mon, 15 Sep 2014 14:02:36 +0200 James Darnley james.darn...@gmail.com wrote: On 2014-09-15 13:17, Clément Bœsch wrote: On Mon, Sep 15, 2014 at 12:46:14AM +0200, James Darnley wrote: --- Still a couple of things on the todo list (see the top of the new file) but I thought I would

Re: [FFmpeg-devel] More specific error codes

2014-09-17 Thread wm4
On Wed, 17 Sep 2014 22:56:27 +0300 Andrey Utkin andrey.krieger.ut...@gmail.com wrote: I have got many complaints about FFmpeg error codes being too abstract. Not helpful for quick resolution, in a lot of such cases real issue with application must be investigated manually (by reviewing log

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 dbdanie...@gmail.com 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

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 u...@pkh.me 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)

[FFmpeg-devel] [PATCH] avformat/mp3dec: fix gapless audio support

2014-09-20 Thread wm4
The code already had skipping of initial padding, but discarding trailing frame padding was missing. This is somewhat questionable, because it will make the decoder discard any data after the declared file size in the LAME header. But note that skipping full frames at the end of the stream is

[FFmpeg-devel] [PATCH] avformat/mp3dec: fix gapless audio support

2014-09-20 Thread wm4
The code already had skipping of initial padding, but discarding trailing frame padding was missing. This is somewhat questionable, because it will make the decoder discard any data after the declared file size in the LAME header. But note that skipping full frames at the end of the stream is

Re: [FFmpeg-devel] [PATCH 11/17] avformat/assenc: mux all extradata at once

2014-09-21 Thread wm4
On Sat, 20 Sep 2014 22:27:51 +0200 Clément Bœsch u...@pkh.me wrote: Before this commit, the code was muxing up to the 2nd line after [Events] (assuming it to be the Format: line). The remaining are generally Comment: directives which can stay in that place. mkvextract behaves that way so it

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

2014-09-21 Thread wm4
On Sat, 20 Sep 2014 22:27:52 +0200 Clément Bœsch u...@pkh.me 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)

Re: [FFmpeg-devel] in preparation of changing the internal form of decoded text subtitles

2014-09-21 Thread wm4
On Sat, 20 Sep 2014 22:27:40 +0200 Clément Bœsch u...@pkh.me wrote: Just a patchset to ease the transition to a sane internal form for the decoded text subtitles. [PATCH 01/17] avcodec/jacosubdec: add some memory checks [PATCH 02/17] avcodec/microdvddec: add some memory checks [PATCH

Re: [FFmpeg-devel] [PATCH 13/17] avformat/assdec: make sure Comments: events end up in the header

2014-09-21 Thread wm4
On Sun, 21 Sep 2014 11:16:49 +0200 Nicolas George geo...@nsup.org wrote: Le jour de la Récompense, an CCXXII, wm4 a écrit : A bit regrettable IMO (more work for a user if a user really wants to get timed comments), but surely more consistent in general (and I guess Comment events got lost

Re: [FFmpeg-devel] [PATCH 2/3] configure: Set large address aware.

2014-09-21 Thread wm4
On Sun, 21 Sep 2014 10:17:15 +0100 Reimar Döffinger reimar.doeffin...@gmx.de wrote: Allows 32 bit version of FFmpeg to use up to 4 GB of RAM instead of just 2 GB on Windows if the OS can provide it. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- configure | 1 + 1 file

[FFmpeg-devel] [PATCH] avformat/mp3dec: avoid early EOF with concatenated gapless mp3s

2014-09-21 Thread wm4
Consider a file created with something like: cat file1.mp3 file2.mp3 result.mp3 Then if file2.mp3 has gapless information, result.mp3 would stop playing something in the middle. This happens because the gapless info directs the decoder to discard all samples after a certain position. To

Re: [FFmpeg-devel] [PATCH] avformat/mp3dec: avoid early EOF with concatenated gapless mp3s

2014-09-21 Thread wm4
On Sun, 21 Sep 2014 15:59:43 +0200 James Darnley james.darn...@gmail.com wrote: On 2014-09-21 12:51, wm4 wrote: it's easy to prevent at least early EOF. Playback will happen to work, even if it's slightly broken. Do we want to print a warning in the slightly broken case? Does it just

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread wm4
On Mon, 22 Sep 2014 14:53:12 + (UTC) Rainer Hochecker fernetme...@online.de wrote: Clément Bœsch u at pkh.me writes: +1 Applied We at XBMC got the first complaints about broken subtitles. Why was this removed? It never worked right in the first place, at least not

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

2014-09-22 Thread wm4
On Mon, 22 Sep 2014 17:13:24 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep 21, 2014 at 10:48:01PM +0200, Jörg Krause wrote: Some C libraries, eg glibc, uclibc, and musl, uses feature test macros to expose definitions conforming to the standards ISO C, POSIX and

  1   2   3   4   5   6   7   8   9   10   >