[FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-07-08 Thread sami . hult
From: Sami Hult This is an - once again - updated patch, that uses avio_write instead of avio_puts to stream clean text output without null characters. Works now for me as intended. Changes metadata filter to accept general urls as file argument without breaking former behaviour. As a byproduct

Re: [FFmpeg-devel] Metadata outputs null-terminated strings

2016-07-08 Thread Sami Hult
On 8 July 2016 at 14:18, Hendrik Leppkes wrote: > On Fri, Jul 8, 2016 at 12:33 PM, Sami Hult wrote: > > Let me rephrase: avio_put_str, which in my patch is used instead of > > (forbidden) printf, outputs the terminating null-character with the > string. > > > >

Re: [FFmpeg-devel] Metadata outputs null-terminated strings

2016-07-08 Thread Sami Hult
Let me rephrase: avio_put_str, which in my patch is used instead of (forbidden) printf, outputs the terminating null-character with the string. On 8 July 2016 at 13:12, Sami Hult wrote: > Hi all, > > metadata filter seems to output null-terminated strings (i.e. > "fram

[FFmpeg-devel] Metadata outputs null-terminated strings

2016-07-08 Thread Sami Hult
Hi all, metadata filter seems to output null-terminated strings (i.e. "frame:14\n\0pts:9\n\0"), which I noticed first when trying to regex-match them. Is this intended behavior and if it is, what is the rationale? Sami -- Mannilantie 37 B 11, 04400 58, tel. +358-46-539-6617

[FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-07-08 Thread sami . hult
From: Sami Hult This is an updated patch, that removes a debug-purposed printf that I accidentally left. Changes metadata filter to accept general urls as file argument without breaking former behaviour. As a byproduct, it also allows for writing to file "-" if specified as "fi

Re: [FFmpeg-devel] [RFC] beginner difficulty bug trac tag for new open source developers

2016-07-04 Thread Sami Hult
On 4 July 2016 at 11:02, Clément Bœsch wrote: > On Sun, Jul 03, 2016 at 02:29:31PM -0400, compn wrote: > [...] > > i'm also not suggesting tagging every bug with a difficulty rating , as > > that would take a while and i'm not sure of the benefit. i just want to > > have a solid number (100-300+)

[FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-06-29 Thread sami . hult
From: Sami Hult NOTE This is a new patch against the latest master branch. It fixes an error in code where "file:-" was directed to file descriptor 2 instead of 1. I changed mode=print file handling of metadata filter to accept any ffmpeg url. This way multiple streams can for

[FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-06-27 Thread sami . hult
From: Sami Hult --- Changelog| 1 + doc/filters.texi | 23 +-- libavfilter/f_metadata.c | 48 3 files changed, 46 insertions(+), 26 deletions(-) diff --git a/Changelog b/Changelog index 4a85925

[FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-06-22 Thread sami . hult
From: Sami Hult I changed output=print file handling of metadata filter to accept any ffmpeg url. This way multiple streams can for example be silencedetected and the result directed to different outputs. Example: ffmpeg -i test.wav -filter_complex "silencedetect=n=-40dB:d=0.1,amet

Re: [FFmpeg-devel] [PATCH] Added pipe output to metadata filter

2016-06-22 Thread Sami Hult
On 22 Jun 2016 16:57, "Paul B Mahol" wrote: > > On 6/22/16, sami.h...@gmail.com wrote: > > From: Sami Hult > > > > I added option mode=pipe which accepts file=(file descriptor) option. This > > way multiple streams can for example be silencedetected

[FFmpeg-devel] [PATCH] Added pipe output to metadata filter

2016-06-22 Thread sami . hult
From: Sami Hult I added option mode=pipe which accepts file=(file descriptor) option. This way multiple streams can for example be silencedetected and the result directed to different outputs. Example: ffmpeg -i test.wav -filter_complex "silencedetect=n=-40dB:d=0.1,ametadata=mode=pipe