Re: [AD] Grabber and vsnprintf

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]

To: alleg-developers@xxxxxxxxxx
Subject: Re: [AD] Grabber and vsnprintf
From: guilt <karthikkumar@xxxxxxxxxx>
Date: Mon, 16 May 2005 10:49:19 +0530
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:in-reply-to:mime-version:content-type:references; b=dSvpAPHeacLH3FqkPJ0bKyNQ7xXS2/oVl1XI5d9U76N884DQzqkh1UcEiC6jInAiPVUI+23Sq6srAVkFNpwMKIJVcZkVcWZWiFlUwykkw4jCVQwPsdYCU18QJUbRd2XKgyuJTXuAVu2GSj0GHi6FGu382f2ryOjyPlAs+cFd9Eg=

I strongly condemn it. the grabber.c doesn't compile in MSVC. i wanted to give a patch that changes all vsnprintfs to vsprintfs (in spite of the obvious hole) >:E

On 5/14/05, Evert Glebbeek <eglebbk@xxxxxxxxxx> wrote:

I'd used vsnprintf a few times in the grabber, but it seems to be C99 only.
The attached patch changes it to uvszprintf, which seems to work just fine
despite not setting the encoding.
There are quite a few calls to vsprintf() in dat that we should probably
also change for good measure.

Evert

--
Karthik
http://guilt.bafsoft.net

Follow-Ups:
- Re: [AD] Grabber and vsnprintf
  - From: Evert Glebbeek
- Re: [AD] Grabber and vsnprintf
  - From: Andrei Ellman

References:
- [AD] Grabber and vsnprintf
  - From: Evert Glebbeek

Messages sorted by: [ date | thread ]
Prev by Date: Re: [AD] No need for END_OF_MAIN (Windows)
Next by Date: [AD] A few fixes to Al-CVS-2005/05/07
Previous by thread: Re: [AD] Grabber and vsnprintf
Next by thread: Re: [AD] Grabber and vsnprintf

Mail converted by MHonArc 2.6.19+

http://listengine.tuxfamily.org/