On 7/2/2024 2:55 PM, Anton Khirnov wrote:
Quoting James Almer (2024-07-02 18:49:36)
Signed-off-by: James Almer <jamr...@gmail.com>
---
  doc/ffmpeg.texi         | 16 ++++++++++++++++
  fftools/ffmpeg.h        | 15 +++++++++++++++
  fftools/ffmpeg_demux.c  | 26 ++++++++++++++++++++++++++
  fftools/ffmpeg_filter.c | 10 ++++++++++
  fftools/ffmpeg_opt.c    | 25 +++++++++++++++++++++++++
  5 files changed, 92 insertions(+)

diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi
index f25f6192eb..f75ed681cf 100644
--- a/doc/ffmpeg.texi
+++ b/doc/ffmpeg.texi
@@ -1262,6 +1262,22 @@ disabled, all output frames of filter graph might not be 
in the same resolution
  and may be inadequate for some encoder/muxer. Therefore, it is not recommended
  to disable it unless you really know what you are doing.
  Disable autoscale at your own risk.
+
+@item -apply_cropping

+@item -apply_cropping[:@var{stream_specifier}] @var{source} 
(@emph{input,per-stream})

Ok.



+Automatically crop the video according to file metadata. Default is @emph{all}.
                                ^
                         after decoding

Ok.

+
+@table @option
+@item none (0)
+Don't apply any cropping metadata.
+@item all (1)
+Apply both codec and container level croppping. This is the default mode.
+@item codec (2)
+Apply codec level croppping.
+@item container (3)
+Apply container level croppping.
+
+@end table
+

Also, this should probably be in the advanced section, since it's marked
as OPT_EXPERT.

So are autoscale and autorotate, which are also outside the advanced section. But i can move it if you prefer.


@@ -715,6 +729,7 @@ AVDictionary *strip_specifiers(const AVDictionary *dict);
  int find_codec(void *logctx, const char *name,
                 enum AVMediaType type, int encoder, const AVCodec **codec);
  int parse_and_set_vsync(const char *arg, int *vsync_var, int file_idx, int 
st_idx, int is_global);
+int parse_and_set_cropping(const char *arg, int *out);

What's the point of this being in a separate file when it's only used
from ffmpeg_demux?

Copy-paste implementation from parse_and_set_vsync(). Will move.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to