[FFmpeg-devel] [PATCH 3/4] lavc/utils: avcodec_string: dump field order when known
Tobias Rapp
t.rapp at noa-archive.com
Thu Oct 6 17:05:46 EEST 2016
On 04.10.2016 06:49, Rodger Combs wrote:
> ---
> libavcodec/utils.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 6f4df93..a0931c6 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -3228,6 +3228,21 @@ void avcodec_string(char *buf, int buf_size, AVCodecContext *enc, int encode)
> av_get_colorspace_name(enc->colorspace));
> }
>
> + if (enc->field_order != AV_FIELD_UNKNOWN) {
> + const char *field_order = "progressive";
> + if (enc->field_order == AV_FIELD_TT)
> + field_order = "top first";
> + else if (enc->field_order == AV_FIELD_BB)
> + field_order = "bottom first";
> + else if (enc->field_order == AV_FIELD_TB)
> + field_order = "top coded first, swapped";
> + else if (enc->field_order == AV_FIELD_BT)
> + field_order = "bottom coded first, swapped";
In the list of comma-separated information tokens it might not be
obvious to the user that ", swapped" semantically belongs to the
preceding "top coded first". What about "top coded first (swapped)" and
"bottom coded first (swapped)"?
Just my personal feeling.
> +
> + av_strlcatf(detail, sizeof(detail), "%s, ", field_order);
> + }
> +
> +
> if (av_log_get_level() >= AV_LOG_DEBUG &&
> enc->chroma_sample_location != AVCHROMA_LOC_UNSPECIFIED)
> av_strlcatf(detail, sizeof(detail), "%s, ",
>
Regards,
Tobias
More information about the ffmpeg-devel
mailing list