From: Kartik Thakore <thakore.kartik@gmail.com>
Date: Fri, 23 Oct 2009 18:26:48 +0000 (-0400)
Subject: Removed insane XS for SDL::get_video_info. Added tests for it in core_video.t
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=20f544ea4f3e7b34c68445aba89ed9c69d411ddf;p=sdlgit%2FSDL_perl.git

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
--- 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
--- 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