[FFmpeg-devel] [PATCH v4] Add 2 timestamp print formats

Nicolas George george at nsup.org
Wed Jul 3 17:52:18 EEST 2019


Ulf Zibis (12019-07-03):
> From 5d62406366560cfab5711120c514a77867bd8c2e Mon Sep 17 00:00:00 2001
> From: Ulf Zibis <Ulf.Zibis at CoSoCo.de>
> Date: 29.06.2019, 17:52:06
> 
> avutil/timestamp: added av_ts2us() and 2 new print formats

The features you add here seem useful, but I wonder: have you considered
the option of making them a single function with a parameter to select
the format? That may make it easier to add new formats later, and have
them supported by any component where this is relevant.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190703/037fd894/attachment.sig>


More information about the ffmpeg-devel mailing list