X-Git-Url: https://git.dogcows.com/gitweb?p=chaz%2Fyoink;a=blobdiff_plain;f=src%2FMoof%2FVideo.cc;h=544e172ad7a00dac196867689e69c8429fd7e0be;hp=eeec47e7eb7d4dcc00b4a6b17fd732f8e61fc93a;hb=be9ebc1104574e5e81e19c5caba0c23b54df826d;hpb=e495074443d9fd7bc16137084cf9de3d031b75c4

diff --git a/src/Moof/Video.cc b/src/Moof/Video.cc
index eeec47e..544e172 100644
--- a/src/Moof/Video.cc
+++ b/src/Moof/Video.cc
@@ -26,11 +26,10 @@
 
 *******************************************************************************/
 
-#include <SDL/SDL_image.h>
-
 #include "Dispatch.hh"
 #include "Engine.hh"
-#include "Exception.hh"
+#include "Error.hh"
+#include "Image.hh"
 #include "Log.hh"
 #include "Settings.hh"
 #include "Video.hh"
@@ -65,9 +64,6 @@ Video::Video(const std::string& caption, const std::string& icon)
 
 void Video::init(const Attributes& attribs)
 {
-	// make sure the engine is initialized before setting up the video
-	Engine::getInstance();
-
 	mContext = 0;
 	mFlags = 0;
 	mAttribs = attribs;
@@ -136,12 +132,12 @@ void Video::setVideoMode(const long mode[3])
 			// on win32, creating a new context via SDL_SetVideoMode will wipe
 			// out the GL state, so we gotta notify everyone to reload their
 			// state after the change
-			Engine::getInstance().dispatch("video.newcontext");
+			engine.dispatch("video.newcontext");
 
 			logInfo("video context recreated");
 #endif
 		}
-		else throw Exception(ErrorCode::SDL_VIDEOMODE);
+		else throw Error(Error::SDL_VIDEOMODE);
 	}
 }
 
@@ -173,12 +169,8 @@ void Video::setIcon()
 {
 	if (mAttribs.icon != "")
 	{
-		SDL_Surface* icon = IMG_Load(mAttribs.icon.c_str());
-		if (icon)
-		{
-			SDL_WM_SetIcon(icon, 0);
-			SDL_FreeSurface(icon);
-		}
+		Image icon(mAttribs.icon);
+		icon.setAsIcon();
 	}
 }
 
@@ -363,6 +355,11 @@ Video::Attributes::Attributes()
 	}
 	settings.get("icon", icon);
 
+	settings.get("fullscreen", fullscreen);
+	settings.get("resizable", resizable);
+	settings.get("showcursor", cursorVisible);
+	settings.get("grab", cursorGrab);
+
 	std::vector<long> dimensions;
 	settings.get("videomode", dimensions);
 	if (dimensions.size() > 1)
@@ -370,12 +367,31 @@ Video::Attributes::Attributes()
 		mode[0] = dimensions[0];
 		mode[1] = dimensions[1];
 	}
-	if (dimensions.size() > 2) mode[2] = dimensions[2];
+	else if (fullscreen)
+	{
+		SDL_Rect** modes = SDL_ListModes(NULL, SDL_FULLSCREEN | SDL_HWSURFACE);
 
-	settings.get("fullscreen", fullscreen);
-	settings.get("resizable", resizable);
-	settings.get("showcursor", cursorVisible);
-	settings.get("grab", cursorGrab);
+		if (modes == (SDL_Rect**)0)
+		{
+			Mf::logError("no native video mode");
+		}
+		else if (modes == (SDL_Rect**)-1)
+		{
+			Mf::logWarning("any resolution allowed; choosing default 800x600");
+			mode[0] = 800;
+			mode[1] = 600;
+		}
+		else
+		{
+			while (*(modes + 1)) ++modes; // skip to the last
+
+			mode[0] = (*modes)->w;
+			mode[1] = (*modes)->h;
+			Mf::logInfo << "choosing native resolution "
+						<< mode[0] << "x" << mode[1] << std::endl;
+		}
+	}
+	if (dimensions.size() > 2) mode[2] = dimensions[2];
 }