From 62da8e7ad84640ab0a2c6d18e97b1f014a9e7bc0 Mon Sep 17 00:00:00 2001 From: "wrenczes@gmail.com" Date: Sat, 9 Apr 2011 21:08:46 +0000 Subject: [PATCH] Minor optimization: wrapped a string fetch routine with #ifdef DEBUG, as the string's not actually used in this context for anything. Tripped across this as the HUDDisplay constantly queries for whether the OSD option is on, which in turn calls this string constructor/destructor on every draw. --- projects/mtg/src/GameOptions.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/projects/mtg/src/GameOptions.cpp b/projects/mtg/src/GameOptions.cpp index 9bc236ecf..49e8c9ebe 100644 --- a/projects/mtg/src/GameOptions.cpp +++ b/projects/mtg/src/GameOptions.cpp @@ -567,7 +567,9 @@ GameOption& GameSettings::operator[](int optionID) GameOption* GameSettings::get(int optionID) { +#ifdef DEBUG string option_name = Options::getName(optionID); +#endif if (optionID < 0) return &invalid_option;