rule_map: improve error message by proper quoting

Our rule names can be arbitrary strings, so improve readability of
error message by properly quoting the rule name; while there, also
properly quote the module name, resulting in better readability if
the module is "".

1 files changed, 4 insertions, 4 deletions

diff --git a/src/buildtool/build_engine/base_maps/rule_map.cpp b/src/buildtool/build_engine/base_maps/rule_map.cpp
index 7bdc14e4..b6f56b2b 100644
--- a/src/buildtool/build_engine/base_maps/rule_map.cpp
+++ b/src/buildtool/build_engine/base_maps/rule_map.cpp
@@ -237,10 +237,10 @@ auto CreateRuleMap(gsl::not_null<RuleFileMap*> const& rule_file_map,
                 auto json_values) {
                 auto rule_it = json_values[0]->find(id.name);
                 if (rule_it == json_values[0]->end()) {
-                    (*logger)(
-                        fmt::format(
-                            "Cannot find rule {} in {}", id.name, id.module),
-                        true);
+                    (*logger)(fmt::format("Cannot find rule {} in {}",
+                                          nlohmann::json(id.name).dump(),
+                                          nlohmann::json(id.module).dump()),
+                              true);
                     return;
                 }