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

basos g noxelia at gmail.com
Mon Dec 29 12:41:24 CET 2008


forwarding to mailing list too.


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).

I chose %y to somehow decrease the length of the snapshot filename.
The only assumption is that one wouldn't mix his/her 1920 fotos with
2020 one's (taken with vlc :). I think thats fine. (e.g. the y2k
incobatibility does not break the purpose that %y was selected for ).

2008/12/26 Rémi Denis-Courmont <rem at videolan.org>:
> Le vendredi 26 décembre 2008, basos g a écrit :
>> reSYN.
>> Oh yes that static buffer..
>> Corrected with localtime_r
>
> The cast in strftime() is highly suspicious. Why do you need an
> ampersand anyway? By the way %y is not Y2K-compliant, which kind of
> sucks (gcc has a warning for that).
>
> --
> Rémi Denis-Courmont
> http://www.remlab.net/
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-Changed-snapshot-naming-format-to-be-prefix-YYMMDD-H(2).patch
Type: text/x-diff
Size: 3188 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20081229/400a389c/attachment.patch>


More information about the vlc-devel mailing list