[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.
Tinglin Liu
tinglin at google.com
Fri Oct 23 20:41:00 CEST 2015
On Thu, Oct 22, 2015 at 4:57 PM, Derek Buitenhuis <
derek.buitenhuis at gmail.com> wrote:
> On 10/23/2015 12:19 AM, wm4 wrote:
> > Wrong, snprintf() always returns the number of characters the string
> > would have been, even if the buffer size is smaller.
>
> That'll teach me to reply past midnight.
>
> I am dumb at night.
>
> > Also, shouldn't this use some av_ wrapper? What about locale issues?
> > Some locales won't print a "." but a ",".
>
> Oh right. C locales.
>
> I don't know if we have a function for this specific problem or not,
> but it would not surprise me if we do.
>
http://stackoverflow.com/questions/3457968/snprintf-simple-way-to-force-as-radix
Here it mentioned using the setlocale() function, but I didn't find any
examples elsewhere though....
Derek, would you do the amend and push? Let me know if you need me to
resend an amended patch. Thanks.
>
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
More information about the ffmpeg-devel
mailing list