widelands-dev team mailing list archive

[Hans Joachim Desserud <launchpad@xxxxxxxxxxxx>]

Hans Joachim Desserud has proposed merging lp:~hjd/widelands/section-initialization into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~hjd/widelands/section-initialization/+merge/242577

Found by scan-build.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~hjd/widelands/section-initialization into lp:widelands.

=== modified file 'src/ui_fsmenu/options.cc'
--- src/ui_fsmenu/options.cc	2014-11-22 10:18:20 +0000
+++ src/ui_fsmenu/options.cc	2014-11-22 22:12:59 +0000
@@ -53,10 +53,8 @@
 };
 
 void add_languages_to_list(UI::Listselect<std::string>* list, const std::string& language) {
-	Section* s = &g_options.pull_section("global");
-
 	Profile ln("txts/languages");
-	s = &ln.pull_section("languages");
+	Section* s = &ln.pull_section("languages");
 	bool own_selected = "" == language || "en" == language;
 
 	// Add translation directories to the list.