[FFmpeg-devel] [PATCH 1/2] avutil: Add av_get_time_base{_q, }() and deprecate AV_TIME_BASE{_Q, }

Michael Niedermayer michaelni at gmx.at
Mon Dec 30 21:30:31 CET 2013

On Mon, Dec 30, 2013 at 07:40:27PM +0000, Derek Buitenhuis wrote:
> On 12/30/2013 4:47 PM, Clément Bœsch wrote:
> > The purpose of the second one was "consistency". I don't mind
> > particularly.
> I would prefer to have both for consistency. Having one but not the other
> violates the principle of least astonishment[1].

fine with me

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131230/fd0283cc/attachment.asc>

More information about the ffmpeg-devel mailing list