[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