Re: [AD] excustom change + d_edit_proc

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


> The attached patch moves the font change code from exgui to excustom,
> and also fixes the d_edit_proc overflow in it.

Ok everywhere.

> Note: dp must point to a buffer at least (d1 + 1) * 6 bytes long. This is
> due to the fact that - depending on the encoding - one character
> can have up to 6 bytes, and you also need room for the terminator
> character.

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."

-- 
Eric Botcazou




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/