From 21ad59bf9e618aeada7305a7ba710672440ab620 Mon Sep 17 00:00:00 2001 From: gfgtdf Date: Fri, 17 Mar 2017 21:58:54 +0100 Subject: [PATCH] add a todo comment --- src/game_initialization/flg_manager.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/game_initialization/flg_manager.cpp b/src/game_initialization/flg_manager.cpp index 6d8f80de5287..a7a5ff8d92d9 100644 --- a/src/game_initialization/flg_manager.cpp +++ b/src/game_initialization/flg_manager.cpp @@ -378,6 +378,9 @@ void flg_manager::update_choosable_factions() } // Sort alphabetically, but with the 'random' option always first + // TODO: some factions liek ageless have 'multiple' random options + // ('any random faction', 'a random default faction', 'a random non-default faction', etc) + // so this code shouldn't assume that there is only one random faction which is on top of the list. std::sort(choosable_factions_.begin() + 1, choosable_factions_.end(), [](const config* c1, const config* c2) { return translation::compare((*c1)["name"].str(), (*c2)["name"].str()) < 0; });