Author: zaroth
Date: Sat Jun 11 00:26:56 2011
New Revision: 49832
URL: http://svn.gna.org/viewcvs/wesnoth?rev=49832&view=rev
Log:
Moved init_video() up to the game_controller_abstract
Modified:
trunk/src/game_controller.cpp
trunk/src/game_controller.hpp
trunk/src/game_controller_abstract.cpp
trunk/src/game_controller_abstract.hpp
trunk/src/game_controller_new.cpp
trunk/src/game_controller_new.hpp
Modified: trunk/src/game_controller.cpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller.cpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller.cpp (original)
+++ trunk/src/game_controller.cpp Sat Jun 11 00:26:56 2011
@@ -277,65 +277,6 @@
}
}
-bool game_controller::init_video()
-{
- if(cmdline_opts_.nogui) {
- if( !(cmdline_opts_.multiplayer || cmdline_opts_.screenshot) ) {
- std::cerr << "--nogui flag is only valid with
--multiplayer flag or --screenshot flag\n";
- return false;
- }
- video_.make_fake();
- game_config::no_delay = true;
- return true;
- }
-
-#if !(defined(__APPLE__))
- surface icon(image::get_image("game-icon.png", image::UNSCALED));
- if(icon != NULL) {
- ///must be called after SDL_Init() and before setting video mode
- ::SDL_WM_SetIcon(icon,NULL);
- }
-#endif
-
- std::pair<int,int> resolution;
- int bpp = 0;
- int video_flags = 0;
-
- bool found_matching = preferences::detect_video_settings(video_,
resolution, bpp, video_flags);
-
- if (cmdline_opts_.bpp) {
- bpp = *cmdline_opts_.bpp;
- } else if (cmdline_opts_.screenshot) {
- bpp = 32;
- }
-
- if(!found_matching) {
- std::cerr << "Video mode " << resolution.first << 'x'
- << resolution.second << 'x' << bpp
- << " is not supported.\n";
-
- if ((video_flags & FULL_SCREEN)) {
- std::cerr << "Try running the program with the
--windowed option "
- << "using a " << bpp << "bpp setting for your
display adapter.\n";
- } else {
- std::cerr << "Try running the program with the
--fullscreen option.\n";
- }
-
- return false;
- }
-
- std::cerr << "setting mode to " << resolution.first << "x" <<
resolution.second << "x" << bpp << "\n";
- const int res =
video_.setMode(resolution.first,resolution.second,bpp,video_flags);
- video_.setBpp(bpp);
- if(res == 0) {
- std::cerr << "required video mode, " << resolution.first << "x"
- << resolution.second << "x" << bpp << " is not
supported\n";
- return false;
- }
-
- return true;
-}
-
bool game_controller::init_config(const bool force)
{
cache_.clear_defines();
Modified: trunk/src/game_controller.hpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller.hpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller.hpp (original)
+++ trunk/src/game_controller.hpp Sat Jun 11 00:26:56 2011
@@ -50,7 +50,6 @@
~game_controller();
bool init_config() { return init_config(false); }
- bool init_video();
bool init_language();
bool play_test();
bool play_multiplayer_mode();
Modified: trunk/src/game_controller_abstract.cpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller_abstract.cpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller_abstract.cpp (original)
+++ trunk/src/game_controller_abstract.cpp Sat Jun 11 00:26:56 2011
@@ -16,6 +16,9 @@
#include "game_controller_abstract.hpp"
#include "game_display.hpp"
+#include "preferences_display.hpp"
+
+#include <iostream>
game_controller_abstract::game_controller_abstract(const commandline_options
&cmdline_opts) :
cmdline_opts_(cmdline_opts),
@@ -35,3 +38,62 @@
}
return *disp_.get();
}
+
+bool game_controller_abstract::init_video()
+{
+ if(cmdline_opts_.nogui) {
+ if( !(cmdline_opts_.multiplayer || cmdline_opts_.screenshot) ) {
+ std::cerr << "--nogui flag is only valid with
--multiplayer flag or --screenshot flag\n";
+ return false;
+ }
+ video_.make_fake();
+ game_config::no_delay = true;
+ return true;
+ }
+
+#if !(defined(__APPLE__))
+ surface icon(image::get_image("game-icon.png", image::UNSCALED));
+ if(icon != NULL) {
+ ///must be called after SDL_Init() and before setting video mode
+ ::SDL_WM_SetIcon(icon,NULL);
+ }
+#endif
+
+ std::pair<int,int> resolution;
+ int bpp = 0;
+ int video_flags = 0;
+
+ bool found_matching = preferences::detect_video_settings(video_,
resolution, bpp, video_flags);
+
+ if (cmdline_opts_.bpp) {
+ bpp = *cmdline_opts_.bpp;
+ } else if (cmdline_opts_.screenshot) {
+ bpp = 32;
+ }
+
+ if(!found_matching) {
+ std::cerr << "Video mode " << resolution.first << 'x'
+ << resolution.second << 'x' << bpp
+ << " is not supported.\n";
+
+ if ((video_flags & FULL_SCREEN)) {
+ std::cerr << "Try running the program with the
--windowed option "
+ << "using a " << bpp << "bpp setting for your
display adapter.\n";
+ } else {
+ std::cerr << "Try running the program with the
--fullscreen option.\n";
+ }
+
+ return false;
+ }
+
+ std::cerr << "setting mode to " << resolution.first << "x" <<
resolution.second << "x" << bpp << "\n";
+ const int res =
video_.setMode(resolution.first,resolution.second,bpp,video_flags);
+ video_.setBpp(bpp);
+ if(res == 0) {
+ std::cerr << "required video mode, " << resolution.first << "x"
+ << resolution.second << "x" << bpp << " is not
supported\n";
+ return false;
+ }
+
+ return true;
+}
Modified: trunk/src/game_controller_abstract.hpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller_abstract.hpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller_abstract.hpp (original)
+++ trunk/src/game_controller_abstract.hpp Sat Jun 11 00:26:56 2011
@@ -31,7 +31,7 @@
game_display& disp();
- virtual bool init_video() = 0;
+ bool init_video();
virtual bool init_config() = 0;
virtual bool init_language() = 0;
virtual bool play_test() = 0;
Modified: trunk/src/game_controller_new.cpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller_new.cpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller_new.cpp (original)
+++ trunk/src/game_controller_new.cpp Sat Jun 11 00:26:56 2011
@@ -30,11 +30,6 @@
}
bool game_controller_new::init_config()
-{
- return true;
-}
-
-bool game_controller_new::init_video()
{
return true;
}
Modified: trunk/src/game_controller_new.hpp
URL:
http://svn.gna.org/viewcvs/wesnoth/trunk/src/game_controller_new.hpp?rev=49832&r1=49831&r2=49832&view=diff
==============================================================================
--- trunk/src/game_controller_new.hpp (original)
+++ trunk/src/game_controller_new.hpp Sat Jun 11 00:26:56 2011
@@ -26,7 +26,6 @@
~game_controller_new();
bool init_config();
- bool init_video();
bool init_language();
bool play_test();
bool play_multiplayer_mode();
_______________________________________________
Wesnoth-commits mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-commits