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

Clément Bœsch ubitux at gmail.com
Sun Oct 21 10:58:40 CEST 2012

On Sun, Oct 21, 2012 at 10:47:56AM +0200, Nicolas George wrote:
> 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.

Since that's the common way we have at the moment for such matter (see git
grep '[0-9]LL'), I'll keep that syntax.

Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121021/26a9fab2/attachment.asc>

More information about the ffmpeg-devel mailing list