On Sun, 11 Feb 2001, Brian S. Julin wrote:

> On Sun, 11 Feb 2001, Christoph Egger wrote:
> 
> > On Sat, 10 Feb 2001, Brian S. Julin wrote:
> > > > /* checks, if an object of the given type and properties is available.
> > > >  * If not, then it fails else it allocates it with the given size.
> > > > */
> > > > ggiOvl_t ggiOvlAllocate(ggi_visual_t vis, enum OverlayFeaturetype type,
> > > >                         void *properties, int width, int height);
> > > 
> > > Question -- why not make the feature type part of the properties?
> > 
> > hmmm... what do you propose?
> 
> Moving the overlay feature type into the structure, so it doesn't have
> to be a parameter.

If I got you right, then ggiOvlAllocate should look like this, no?


/* 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);


> Since you are in the early ALPHA stages, don't hesitate to rework
> the API even after a few releases if you think of something that
> should be done differently/better.  My wood shop teacher taught
> me two important lessons in high school.  The first was "measure
> twice, cut once".  You just did that part by spending a good
> amount of time asking for suggestions on the list.  The second
> lesson was that no matter how carefully you plane a peice of
> wood, you'll always get a better result if you plane it twice at
> half the depth.

Well, I haven't done too much work yet. So I can change it easily.


CU,

Christoph Egger
E-Mail: [EMAIL PROTECTED]


Reply via email to