[FFmpeg-devel] [PATCH]ffmpeg: Do not print "SDL:" on top of sdl output file

Michael Niedermayer michael at niedermayer.cc
Mon Jan 27 01:45:43 EET 2020


On Sun, Jan 26, 2020 at 09:57:59PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch is supposed to fix ticket #7068.
> 
> Please comment, Carl Eugen

>  ffmpeg.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 5a59a8ef62b4bfcde812f8ce81aee5c0f39c02db  0001-ffmpeg-Do-not-print-SDP-on-top-of-sdp-files.patch
> From ab80ef51e9ea87a3d3d0796aabd23b4bef85a4c3 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg at gmail.com>
> Date: Sun, 26 Jan 2020 21:47:26 +0100
> Subject: [PATCH] ffmpeg: Do not print "SDP:" on top of sdp files.
> 
> Fixes ticket #7068.
> ---
>  fftools/ffmpeg.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
> index 6bcd7b94d2..d4ff30d6fb 100644
> --- a/fftools/ffmpeg.c
> +++ b/fftools/ffmpeg.c
> @@ -2776,7 +2776,7 @@ static void print_sdp(void)
>          if (avio_open2(&sdp_pb, sdp_filename, AVIO_FLAG_WRITE, &int_cb, NULL) < 0) {
>              av_log(NULL, AV_LOG_ERROR, "Failed to open sdp file '%s'\n", sdp_filename);
>          } else {
> -            avio_printf(sdp_pb, "SDP:\n%s", sdp);
> +            avio_printf(sdp_pb, "%s", sdp);

Are there any scripts/sw which expects this SDP prefix ?
if not then this patch should be ok

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200127/00cbadd9/attachment.sig>


More information about the ffmpeg-devel mailing list