[mythtv] Ticket #7759

Daniel Kristjansson danielk at cuymedia.net
Sun Dec 20 15:44:04 UTC 2009


On Sun, 2009-12-20 at 15:45 +1100, Davin McCall wrote:

> |2566:         repeat_delay = frame_int_erval * buffer->repeat_pict * 
> 0.5;_|

>      repeat_delay = frame_interval / 2 * buffer->repeat_pict;

I'm revoking your programmers license :) You should never ever use a
division when a multiply will do. Maybe >>1 is what you meant to
say? It avoids the conversion from int to double that 0.5 forces.

-- Daniel



More information about the mythtv-dev mailing list