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/