[FFmpeg-devel] [PATCH] bprint: implement vsnprintf for win32.
Martin Sliwka
martin.sliwka at gmail.com
Fri Jun 15 19:50:47 CEST 2012
Dne 15.6.2012 13:37, Nicolas George napsal(a):
> L'octidi 28 prairial, an CCXX, Alexander Strasser a écrit :
>> sorry for joining in late. I did not look up what this is
>> about. But IMHO the commit message was definitely lacking.
>> I apologize if this was pushed with a better message.
>>
>> It is very good to summarize the change as you did in the
>> subject, but in this case you IMHO definitely need to add a
>> larger message body explaining the motivation and why this
>> should fix the problem you noticed (on FATE).
>>
>> This is not meant to discourage you or anyone, I do very
>> much appreciate your work! Also talk is cheap and I know
>> I am late to complain, so please forgive my whining.
> No problem. I hesitated to be more verbose in the commit message. If you
> want, I can add a comment near the additional code:
>
> /* Windows's snprintf returns -1 if the buffer is not large enough,
> the standard requires the number of bytes that would have been used.
> _vscprintf computes that number of bytes. */
>
> If you agree and no one objects, I can push it directly.
>
> Regards,
>
I have problem cross-compiling using MinGW:
/root/ffmpeg/git/libavutil/bprint.c: In function 'vsnprintf_fixed':
/root/ffmpeg/git/libavutil/bprint.c:40: error: implicit declaration of
function '_vscprintf'
Regards,
Martin
More information about the ffmpeg-devel
mailing list