async_map_utils: Pass key_printer also for reporting pending tasks...

...in async map instances, same as for reporting cycles.

This removes the restriction that the key object has to posses the
ToString method, allowing it to be used, e.g., with just-mr maps.

The now obsolete HasToString concept is removed.

1 files changed, 6 insertions, 3 deletions

diff --git a/src/buildtool/main/analyse.cpp b/src/buildtool/main/analyse.cpp
index 5783114c..c9d61872 100644
--- a/src/buildtool/main/analyse.cpp
+++ b/src/buildtool/main/analyse.cpp
@@ -198,9 +198,12 @@ namespace Target = BuildMaps::Target;
             Logger::Log(logger, LogLevel::Error, *error_msg);
             return std::nullopt;
         }
-        DetectAndReportPending("expressions", expr_map, logger);
-        DetectAndReportPending("rules", rule_map, logger);
-        DetectAndReportPending("targets", target_map, logger);
+        DetectAndReportPending(
+            "expressions", expr_map, Base::kEntityNamePrinter, logger);
+        DetectAndReportPending(
+            "rules", rule_map, Base::kEntityNamePrinter, logger);
+        DetectAndReportPending(
+            "targets", target_map, Target::kConfiguredTargetPrinter, logger);
         return std::nullopt;
     }