[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