| Message ID | 1549781638-1099-2-git-send-email-mypopydev@gmail.com |
|---|---|
| State | New |
| Headers | show |
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 <mypopydev@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 [...]
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)); } av_packet_unref(&pkt); av_init_packet(&pkt);
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 <mypopydev@gmail.com> --- tests/api/api-h264-test.c | 12 +++++++++--- 1 files changed, 9 insertions(+), 3 deletions(-)