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