AI code: remove old version annotations from TODOs

Also use consistent capitalization and add a couple comments referring to existing TODOs.

src/ai/contexts.cpp

@@ -384,7 +384,7 @@ void readonly_context_impl::calculate_moves(const unit_map& units, std::map<map_
 			dstsrc.insert(trivial_mv);
 		}
 		/**
-		 * @todo This is where support for a speculative unit map is incomplete.
+		 * TODO: This is where support for a speculative unit map is incomplete.
 		 *       There are several places (deep) within the paths constructor
 		 *       where resources::gameboard->units() is assumed to be the unit map. Rather
 		 *       than introduce a new parameter to numerous functions, a better

src/ai/contexts.hpp

@@ -1260,7 +1260,7 @@ public:
 	 * use of a speculative unit map.
 	 * NOTE: Support for a speculative map is broken (not used when pathfinding)
 	 *       and has not been used since (probably) r38610 (September 2009).
-	 *       (See the todo in the implementation.)
+	 *       (See the TODO in the implementation.)
 	 */
 	void calculate_moves(const unit_map& units,
 		std::map<map_location,pathfind::paths>& possible_moves, move_map& srcdst,

src/ai/default/contexts.hpp

@@ -285,7 +285,7 @@ public:
 
 private:
 	recursion_counter recursion_counter_;
-	mutable std::vector<target> additional_targets_;///@todo 1.9 refactor this
+	mutable std::vector<target> additional_targets_;// TODO: refactor this (remove mutable)
 
 
 };

src/ai/lua/engine_lua.cpp

@@ -377,7 +377,7 @@ void engine_lua::do_parse_goal_from_config(const config &cfg, std::back_insert_i
 
 std::string engine_lua::evaluate(const std::string &/*str*/)
 {
-	///@todo this is not mandatory, but if we want to allow lua to evaluate
+	// TODO: this is not mandatory, but if we want to allow lua to evaluate
 	// something 'in context' of this ai, this will be useful
 	return "";
 }

src/ai/manager.cpp

@@ -490,7 +490,7 @@ bool manager::should_intercept( const std::string& str ) const
 //this is stub code to allow testing of basic 'history', 'repeat-last-command', 'add/remove/replace ai' capabilities.
 //yes, it doesn't look nice. but it is usable.
 //to be refactored at earliest opportunity
-///@todo 1.9 extract to separate class which will use fai or lua parser
+// TODO: extract to separate class which will use fai or lua parser
 const std::string manager::internal_evaluate_command( side_number side, const std::string& str ){
 	const int MAX_HISTORY_VISIBLE = 30;
 

src/ai/manager.hpp

@@ -15,8 +15,8 @@
 /**
  * @file
  * Managing the AIs lifecycle - headers
- * @todo 1.9 Refactor history handling and internal commands.
- * @todo 1.9 AI Interface command to clear the history.
+ * TODO: Refactor history handling and internal commands.
+ * TODO: AI Interface command to clear the history.
  */
 
 #pragma once
@@ -91,7 +91,7 @@ private:
 };
 
 /**
- * AI Command History Item. It is an implementation detail
+ * AI Command History Item. It is an implementation detail. See TODOs above.
  */
 class command_history_item{
 public:
@@ -484,7 +484,7 @@ private:
 	 * @param side side number (1-based).
 	 * @param str string to evaluate.
 	 * @return string result of evaluation.
-	 * @todo 1.9 rewrite this function to use a fai or lua parser.
+	 * TODO: rewrite this function to use a fai or lua parser.
 	 */
 	const std::string internal_evaluate_command( side_number side, const std::string& str );