From: ffmpegagent <ffmpegagent@gmail.com>
To: ffmpeg-devel@ffmpeg.org
Cc: softworkz <softworkz@hotmail.com>
Subject: [FFmpeg-devel] [PATCH 0/2] [RFC-NotForMerge] fftools/graphprint: The "bad" patch
Date: Wed, 28 May 2025 15:24:07 +0000
Message-ID: <pull.89.ffstaging.FFmpeg.1748445849.ffmpegagent@gmail.com> (raw)
This is the "bad" patch from the graphprint patchset. It is largely
unchanged, except:
* The method to determine the temp folder has been revised
* Only xdg-open is called for Linux
(no alternate aliases)
* An additional check is added for illustration
(but not actually necessary)
A second patch is included which replaces the system() with the original
source code from glibc 2.31. This is for being able to actually discuss the
code rather than a "black box".
Please see my accompanying e-mail: The "bad" Patch
.
softworkz (2):
[RFC-NotForMerge] fftools/graphprint: The "bad" patch
[RFC-NotForMerge] fftools/graphprint: Inline glibc system()
implementation
doc/ffmpeg.texi | 4 +
fftools/Makefile | 2 +
fftools/ffmpeg.c | 2 +-
fftools/ffmpeg.h | 1 +
fftools/ffmpeg_filter.c | 2 +-
fftools/ffmpeg_opt.c | 4 +
fftools/graph/_system.c | 213 ++++++++++++++++++++++++++++++++
fftools/graph/_system.h | 27 ++++
fftools/graph/_system_support.h | 199 +++++++++++++++++++++++++++++
fftools/graph/filelauncher.c | 201 ++++++++++++++++++++++++++++++
fftools/graph/graphprint.c | 48 ++++++-
fftools/graph/graphprint.h | 32 +++++
12 files changed, 732 insertions(+), 3 deletions(-)
create mode 100644 fftools/graph/_system.c
create mode 100644 fftools/graph/_system.h
create mode 100644 fftools/graph/_system_support.h
create mode 100644 fftools/graph/filelauncher.c
base-commit: 96d4bcbcd81f18e4538f139fb86d52d41981c4be
Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-89%2Fsoftworkz%2Fkillerfeature_system-v1
Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-ffstaging-89/softworkz/killerfeature_system-v1
Pull-Request: https://github.com/ffstaging/FFmpeg/pull/89
--
ffmpeg-codebot
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next reply other threads:[~2025-05-28 15:24 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-28 15:24 ffmpegagent [this message]
2025-05-28 15:24 ` [FFmpeg-devel] [PATCH 1/2] " softworkz
2025-05-28 15:24 ` [FFmpeg-devel] [PATCH 2/2] [RFC-NotForMerge] fftools/graphprint: Inline glibc system() implementation softworkz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pull.89.ffstaging.FFmpeg.1748445849.ffmpegagent@gmail.com \
--to=ffmpegagent@gmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
--cc=softworkz@hotmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git