> > /* checks, if an object of the given properties is available.
> >  * If not, then it fails else it allocates it with the given size.
> >  */
> > ggiOvl_t ggiOvlAllocate(ggi_visual_t vis,
> >                     struct properties *props,
> >                     int width, int height);

> Yes -- but what's a struct properties?  Any reason why it shouldn't 
> be just a ggi_Ovl_t?

Cause you need to access it. I'd like it better, if the ggi_Ovl_t
would be an opaque handle ... will keep people from doing strange things
with internals.

Even if you do it like:

ggi_Ovl_t {
        /* public */
        int some prop;
        ...
        /* don't touch this */
        void *priv;
}

people will fiddle with it I bet. And then wonder why their stuff breaks in
later releases or other targets.

CU, Andy

-- 
Andreas Beck              |  Email :  <[EMAIL PROTECTED]>

Reply via email to