On Fri, 2004-02-13 at 09:57, Jens Rieks wrote:

> here is an easy to use (with PIR code) wrapper of the SDL library.
> It tries to hide all internals of the wrapper and makes the most important
> SDL functions directly available in PIR code.

Overall this looks good.  I am not sure if it is better to define flags
as constants or to do something like this:

        .sym pmc flags

        new flags, .PerlHash
        set flags['double_buffered'], 1
        set flags['full_screen'], 1

        .sym pmc screen
        screen = _SDL_SetVideoMode( 320, 200, 0, flags )

Then only _SDL_SetVideoMode has to care about the flags.

It is an idea anyway.  Another idea is to pass similar arguments into
_SDL_init_flags() and receive back an integer.  I prefer to hide that in
_SDL_SetVideoMode() though.

-- c

Reply via email to