[FFmpeg-devel] [patch 3/3] Make timing calculations less dependant on start_time being defined.

[Michael Niedermayer]

Hi

On Wed, Aug 22, 2007 at 07:24:48PM +0200, Andreas ?man wrote:
> Hello,
>
> Michael Niedermayer wrote:
>> the functions name no longer matches what it does ...
>> has_duration() would make more sense
>
> ...
>
>> tabs are forbidden in ffmpeg svn
>
> ...
>
> Since Neil does not seem to come back with a cleaned up patch
> I've done that. (Getting invalid durations bugs me a bit :)
[...]
> @@ -1403,17 +1404,26 @@
>                      end_time = end_time1;
>              }
>          }
> +        if (st->duration != AV_NOPTS_VALUE) {
> +                duration1 = av_rescale_q(st->duration, st->time_base, AV_TIME_BASE_Q);
> +                if (duration1 > duration)
> +                        duration = duration1;
> +        }

indention should be 4 spaces ...

besides this the patch looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070823/e8f4819e/attachment.pgp>