Removed insane XS for SDL::get_video_info. Added tests for it in core_video.t

diff --git a/src/Core/Video.xs b/src/Core/Video.xs
index 4016420..289d749 100644 (file)
--- a/src/Core/Video.xs
+++ b/src/Core/Video.xs
@@ -30,34 +30,11 @@ video_get_video_surface()
                RETVAL
 
 
-HV *
+SDL_VideoInfo*
 video_get_video_info()
+       PREINIT:
+               char* CLASS = "SDL::VideoInfo";
        CODE:
-               HV *hv;
-               SDL_VideoInfo *info;
-               info = (SDL_VideoInfo *) safemalloc ( sizeof(SDL_VideoInfo));
-               memcpy(info,SDL_GetVideoInfo(),sizeof(SDL_VideoInfo));
-               hv = newHV();
-               hv_store(hv,"hw_available",strlen("hw_available"),
-                       newSViv(info->hw_available),0);
-               hv_store(hv,"wm_available",strlen("wm_available"),
-                       newSViv(info->wm_available),0);
-               hv_store(hv,"blit_hw",strlen("blit_hw"),
-                       newSViv(info->blit_hw),0);
-               hv_store(hv,"blit_hw_CC",strlen("blit_hw_CC"),
-                       newSViv(info->blit_hw_CC),0);
-               hv_store(hv,"blit_hw_A",strlen("blit_hw_A"),
-                       newSViv(info->blit_hw_A),0);
-               hv_store(hv,"blit_sw",strlen("blit_sw"),
-                       newSViv(info->blit_sw),0);
-               hv_store(hv,"blit_sw_CC",strlen("blit_sw_CC"),
-                       newSViv(info->blit_sw_CC),0);
-               hv_store(hv,"blit_sw_A",strlen("blit_sw_A"),
-                       newSViv(info->blit_sw_A),0);
-               hv_store(hv,"blit_fill",strlen("blit_fill"),
-                       newSViv(info->blit_fill),0);
-               hv_store(hv,"video_mem",strlen("video_mem"),
-                       newSViv(info->video_mem),0);
-               RETVAL = hv;
+               RETVAL = SDL_GetVideoInfo;
                OUTPUT:
-               RETVAL
+       RETVAL

diff --git a/t/core_video.t b/t/core_video.t
index fe3d5d7..52b4d0b 100644 (file)
--- a/t/core_video.t
+++ b/t/core_video.t
@@ -5,7 +5,7 @@ use SDL::Config;
 
 use Test::More;
 
-plan ( tests => 2 );
+plan ( tests => 4 );
 
 use_ok( 'SDL::Video' ); 
   
@@ -21,6 +21,8 @@ my $display = SDL::SetVideoMode(640,480,32, SDL_SWSURFACE );
 
 isa_ok(SDL::Video::get_video_surface(), 'SDL::Surface', 'Checking if we get a surface ref back'); 
 
+isa_ok(SDL::Video::get_video_info(), 'SDL::VideoInfo', 'Checking if we get videoinfo ref back');
+
 =skip
        video_driver_name
        list_modes