[mythtv] Re: [mythtv-commits] mythtv commit: r7587 - in trunk/mythtv/libs bydanielk

[Daniel Kristjansson]

On Wed, 2005-10-26 at 10:11 +1000, Mark Spieth wrote:
> > New Revision: 7587

> I probably would have used an intermediate for the av_q2d function so the
> divide didnt need to be done twice.
I thought about it; but this is only done on stream changes,
so I went with the option that looked more readable to me.

> also wass unsure of the impact on h263 hence the check for a large den.
Looks reasonable to me, it could cause a problem with some very low
fps h263 streams (cell-phones?), but we'll cross that bridge when we
get to it.

-- Daniel