From: Marton Balint <cus@passwd.hu>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 05/10] doc/muxers: Add documentation for segment_limit option
Date: Fri, 13 Jun 2025 22:25:35 +0200 (CEST)
Message-ID: <beff2f84-a860-7d7e-fea3-c15bb8959a7b@passwd.hu> (raw)
In-Reply-To: <53b693320f1cf07ee6094342a60002bc9cfc3d79.1749794067.git.ffmpegagent@gmail.com>
On Fri, 13 Jun 2025, softworkz wrote:
> From: softworkz <softworkz@hotmail.com>
>
> Signed-off-by: softworkz <softworkz@hotmail.com>
> ---
> doc/muxers.texi | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/doc/muxers.texi b/doc/muxers.texi
> index 6d5c17b4cc..1cca8da1fb 100644
> --- a/doc/muxers.texi
> +++ b/doc/muxers.texi
> @@ -3510,6 +3510,13 @@ packet written. Defaults to @code{0}.
> Write segments to files with a .tmp extension. Each file is renamed to its
> actual name on completion. This can help to prevent segment files from
> being accessed before they are complete. Disabled by default (@code{0}).
> +
> +@item segment_limit @var{number}
> +Stops after the specified number of segments has been generated.
> +This can be helpful to fill gaps in a range of already generated segments,
> +which is difficult to achieve otherwise as it would either cause the last
> +segment to be incomplete or to overwrite an existing segment with a partial
> +data. Default is @code{0} - no limit.
You can merge the documentation patch with the feature patch, there is no
need to split.
What is not quite clear is that what is going to happen to the surplus
data at after the last segment? Is it silently dropped? Because that would
be unacceptable IMHO.
Regards,
Marton
_______________________________________________
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 prev parent reply other threads:[~2025-06-13 20:28 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-13 5:54 [FFmpeg-devel] [PATCH 00/10] avformat/segment: Various segment muxer improvements ffmpegagent
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 01/10] avformat/segment: Add segment_write_temp option softworkz
2025-06-13 20:37 ` Marton Balint
2025-06-13 21:43 ` softworkz .
2025-06-13 22:07 ` Marton Balint
2025-06-13 22:26 ` softworkz .
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 02/10] doc/muxers: Add documentation for " softworkz
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 03/10] avformat/segment: Log more detailed information about written segments softworkz
2025-06-13 13:42 ` Derek Buitenhuis
2025-06-13 14:03 ` softworkz .
2025-06-13 20:44 ` Marton Balint
2025-06-13 23:53 ` softworkz .
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 04/10] avformat/segment: Add segment_limit option softworkz
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 05/10] doc/muxers: Add documentation for " softworkz
2025-06-13 20:25 ` Marton Balint [this message]
2025-06-13 21:33 ` softworkz .
2025-06-13 21:59 ` Marton Balint
2025-06-13 22:49 ` softworkz .
2025-06-14 15:59 ` Marton Balint
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 06/10] avformat/segment: Fix invalid codecpar extradata_size after copying softworkz
2025-06-13 6:03 ` softworkz .
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 07/10] avformat/segment: Remove non-negative constraint from segment_time_delta softworkz
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 08/10] avformat/segment: Don't allow negative segment duration softworkz
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 09/10] avformat/segment: Fix typo softworkz
2025-06-13 5:54 ` [FFmpeg-devel] [PATCH 10/10] avformat/segment: Indent and whitespace fixes softworkz
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 0/6] avformat/segment: Various segment muxer improvements ffmpegagent
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 1/6] avformat/segment: Add segment_write_temp option softworkz
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 2/6] avformat/segment: Add segment_limit option softworkz
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 3/6] avformat/segment: Remove non-negative constraint from segment_time_delta softworkz
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 4/6] avformat/segment: Don't allow negative segment duration softworkz
2025-06-14 20:40 ` Michael Niedermayer
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 5/6] avformat/segment: Fix typo softworkz
2025-06-14 0:59 ` [FFmpeg-devel] [PATCH v2 6/6] avformat/segment: Indent and whitespace fixes 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=beff2f84-a860-7d7e-fea3-c15bb8959a7b@passwd.hu \
--to=cus@passwd.hu \
--cc=ffmpeg-devel@ffmpeg.org \
/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