Andreas Beck <[EMAIL PROTECTED]> writes:

> display/X already contains mode.inc, which is shared by the various X
> targets.

has anyone looked at this code recently?  besides not being entirely
correct, the naming of the following functions is inconsistent.
should i fix this as well when i add make_cursor() here?

 *   int _ggi_x_resize(ggi_visual_t vis, int w, int h, ggi_event *ev);

 *   static int _GGIgetbpp(ggi_visual *vis);
     this is actually static int _GGIgetbpp(ggi_x_priv *priv)

 *   static int _GGIbasiccheck(ggi_visual *vis,ggi_mode *tm,
                               XVisualInfo *vinfo);

 *   int GGI_X_checkmode(ggi_visual *vis,ggi_mode *tm);

 *   int GGI_X_setmode(ggi_visual *vis,ggi_mode *tm);

 *   int GGI_X_getmode(ggi_visual *vis,ggi_mode *tm);

there seem to be other non-documented functions as well.  what was the
intention of listing these 5?  historic reasons? :-)

which of GGI_X, _ggi_x, _GGI is preferred?

-- 
Tijs van Bakel, <[EMAIL PROTECTED]>

Reply via email to