On Sun, Feb 10, 2019 at 02:53:58PM +0800, Jun Zhao wrote:
> Add AV_NOPTS_VALUE check for AVFrame.pkt_dts to avoid print the
> pkt_dts as negative number like:
> "0, 3616613, -9223372036854775808, 1001, 3110400, 0x75e37a65"
>
> Signed-off-by: Jun Zhao <mypopy...@gmail.com>
> ---
> tests/api/api-h264-test.c | 12 +++++++++---
> 1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/tests/api/api-h264-test.c b/tests/api/api-h264-test.c
> index 66669fa..351d622 100644
> --- a/tests/api/api-h264-test.c
> +++ b/tests/api/api-h264-test.c
> @@ -131,9 +131,15 @@ static int video_decode_example(const char
> *input_filename)
> av_log(NULL, AV_LOG_ERROR, "Can't copy image to
> buffer\n");
> return number_of_written_bytes;
> }
> - printf("%d, %10"PRId64", %10"PRId64", %8"PRId64", %8d,
> 0x%08lx\n", video_stream,
> - fr->pts, fr->pkt_dts, fr->pkt_duration,
> - number_of_written_bytes, av_adler32_update(0, (const
> uint8_t*)byte_buffer, number_of_written_bytes));
> +
> + if (fr->pkt_dts == AV_NOPTS_VALUE)
> + printf("%d, %10"PRId64", %s, %8"PRId64", %8d,
> 0x%08lx\n", video_stream,
> + fr->pts, "N/A", fr->pkt_duration,
> + number_of_written_bytes, av_adler32_update(0,
> (const uint8_t*)byte_buffer, number_of_written_bytes));
> + else
> + printf("%d, %10"PRId64", %10"PRId64", %8"PRId64", %8d,
> 0x%08lx\n", video_stream,
> + fr->pts, fr->pkt_dts, fr->pkt_duration,
> + number_of_written_bytes, av_adler32_update(0,
> (const uint8_t*)byte_buffer, number_of_written_bytes));you can split the printf() in 2 so the common parts are not duplicated [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The smallest minority on earth is the individual. Those who deny individual rights cannot claim to be defenders of minorities. - Ayn Rand
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
