[vlc-devel] [PATCH] Changed snapshot naming format to be prefix-YYMMDD-HHmms.ext

basos g noxelia at gmail.com
Mon Dec 29 14:24:25 CET 2008


2008/12/29 Rémi Denis-Courmont <rem at videolan.org>:
> Le lundi 29 décembre 2008, basos g a écrit :
>> That's right. We don't need the pointer. The funny thing is that it
>> did'n crash ( probably because it is a local var).
>
> The use of stat() is quite silly, here. I believe two thread checking it
> at about the same time, they'll use the same filename anyhow.
>

This is to account for the case that snapshots are taken at a rate
higher than 1 per second. It assures that not snap will be lost due to
fast snap rate or even (but less possible) for filenames that already
exists ...

I considered it an issue and i would implement it that way. If you
don't i won't pose another veto, though...



More information about the vlc-devel mailing list