On 08/03/2022 00:17, Michael Niedermayer wrote:
Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
---
doc/bitstream_filters.texi | 15 ++++
libavcodec/Makefile | 1 +
libavcodec/bitstream_filters.c | 1 +
libavcodec/dv_error_marker_bsf.c | 117 +++++++++++++++++++++++++++++++
4 files changed, 134 insertions(+)
create mode 100644 libavcodec/dv_error_marker_bsf.c
diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
index a0092878c8..6a882ade97 100644
--- a/doc/bitstream_filters.texi
+++ b/doc/bitstream_filters.texi
@@ -132,6 +132,21 @@ the header stored in extradata to the key packets:
ffmpeg -i INPUT -map 0 -flags:v +global_header -c:v libx264 -bsf:v dump_extra
out.ts
@end example
+@section dv_error_marker
+
+Blocks in DV which are marked as damaged are replaced by blocks of the
specified color.
+
+@table @option
+@item color
+The color to replace damaged blocks by
+@item sta
+The error status to replace. If -1 then the stamask is used. -1 is the default.
+@item stamask
+A 16 bit mask which specifies which of the 16 possible error status values are
+to be replaced by colored blocks. 0xFFFE is the default which replaces all non 0
+error status values.
+@end table
+
@section eac3_core
[...]
In my opinion it would be more user-friendly to use a single "status"
option of AV_OPT_TYPE_FLAGS with "s0" ... "s15" constants, instead of
the "sta"/"stamask" option pair. Maybe there also could be constants for
hard errors (STA = 0111 / 1111) and error concealment types A/B/C to
make the option value readable.
Also it would be nice to add the link to S314M documentation pages also
in the option documentation.
Regards,
Tobias
_______________________________________________
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".
