> But to allow to specify the properties of VRAM/Ovl as you suggested I
> added this one:
> 
> int ggiOvlCheck(ggi_visual_t vis, enum Overlaytype type,
>               void *properties, int width, int height);
> 
> Its behaviour is similar to ggiCheckMode();

O.K. - but it needs to be added to the Allocate as well.

- /* checks, if a area of VRAM is available. If not, then it fails
+ /* 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.
-  * else it allocates it with the given size.
>  */
> ggiOvl_t ggiOvlAllocate(ggi_visual_t vis, enum Overlaytype type,
+                       void *props, int width, int height);
-                       int width, int height);

> /* checks, if the given type of overlay with the given properties
>  * will work. If not, then the modified properties passed back
>  * should work. If NULL is passed back, then NOTHING would work

You cannot pass back NULL. You can modify the contents of the struct, not
the pointer. Do this with a failure code in the return code.

>  * on the current target.
>  */
> int ggiOvlCheck(ggi_visual_t vis, enum Overlaytype type,
>                 void *properties, int width, int height);

CU, ANdy

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

Reply via email to