0ct0pu5/wesnoth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixup silenced warnings in ai/actions.cpp

Browse source 
Better to demote them to a lower channel rather than disable
completely.
This commit is contained in:
Chris Beck 2014-06-29 13:49:11 -04:00
parent 164d7161fc
commit 7185d6cc80
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/ai/actions.cpp
View file

@ -92,7 +92,7 @@ void action_result::execute()
try { try {
do_execute(); do_execute();
} catch (end_level_exception&) { } catch (end_level_exception&) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -296,7 +296,7 @@ void attack_result::do_execute()
try { try {
manager::raise_gamestate_changed(); manager::raise_gamestate_changed();
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -471,7 +471,7 @@ void move_result::do_execute()
try { try {
manager::raise_gamestate_changed(); manager::raise_gamestate_changed();
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -616,7 +616,7 @@ void recall_result::do_execute()
try { try {
manager::raise_gamestate_changed(); manager::raise_gamestate_changed();
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -757,7 +757,7 @@ void recruit_result::do_execute()
try { try {
manager::raise_gamestate_changed(); manager::raise_gamestate_changed();
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -854,7 +854,7 @@ void stopunit_result::do_execute()
manager::raise_gamestate_changed();//to be on the safe side manager::raise_gamestate_changed();//to be on the safe side
} }
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }
@ -905,7 +905,7 @@ void synced_command_result::do_execute()
set_gamestate_changed(); set_gamestate_changed();
manager::raise_gamestate_changed(); manager::raise_gamestate_changed();
} catch (...) { } catch (...) {
is_ok(); //Silences "unchecked result" warning if (!is_ok()) { DBG_AI_ACTIONS << "Return value of AI ACTION was not checked. This may cause bugs! " << std::endl; } //Demotes to DBG "unchecked result" warning
throw; throw;
} }
} }