Re: [AD] excustom change + d_edit_proc |
[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]
Eric Botcazou wrote:
The attached patch moves the font change code from exgui to excustom, and also fixes the d_edit_proc overflow in it.Ok everywhere.
Commited (also the grabber patch).
I find "This is due to the fact that" not very elegant. Why not rather: "Note: dp must point to a buffer at least (d1 + 1) * 6 bytes long because, depending on the encoding format in use, a single character can occupy up to 6 bytes and room must be reserved for the terminating NULL character."
Sure. Just about the "NULL" - is there really a NULL-pointer as well as a NULL-character? I always found this confusing, because they are two different things.. OTOH, str[strlen (str)] = NULL; works, so I guess it makes sense..
-- Elias Pschernig <elias@xxxxxxxxxx>
Mail converted by MHonArc 2.6.19+ | http://listengine.tuxfamily.org/ |