[FFmpeg-devel] [PATCH 1/3] lavf/srtdec: simplify start/end computation.

Nicolas George nicolas.george at normalesup.org
Sun Oct 21 10:47:56 CEST 2012


You replied in private again. There may be something wrong in your settings
that makes you do that frequently.

Le decadi 30 vendémiaire, an CCXXI, Clément Bœsch a écrit :
> I thought the long long type was guaranteed to be at least 64 bits by the
> c99 standard…

It can not be smaller, but it can be larger, and possibly less efficient.

> > hh1 * (int64_t)3600 + ...
> looks ugly :(

I find the LL ugly too. This is a matter of taste. Of course, in the end, it
is your patch, you decide.

> Also we use the LL in various places for that purpose.

I would have complained if I had spotted them at the time.

Regards,

-- 
  Nicolas George
-------------- 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/20121021/4215c5af/attachment.asc>


More information about the ffmpeg-devel mailing list