simplify and use chomp()

1 files changed, 2 insertions, 8 deletions

diff --git a/searchlib/src/vespa/searchlib/fef/blueprintresolver.cpp b/searchlib/src/vespa/searchlib/fef/blueprintresolver.cpp
index eb1876da45c..864dd64b98f 100644
--- a/searchlib/src/vespa/searchlib/fef/blueprintresolver.cpp
+++ b/searchlib/src/vespa/searchlib/fef/blueprintresolver.cpp
@@ -134,14 +134,8 @@ struct Compiler : public Blueprint::DependencyHandler {
             failed_set.insert(feature_name);
             auto trace = make_trace(skip_self);
             vespalib::string msg;
-            if (trace.empty()) {
-                msg = fmt("invalid %s: %s", describe(feature_name).c_str(), reason.c_str());
-            } else {
-                msg = fmt("invalid %s: %s\n%s", describe(feature_name).c_str(), reason.c_str(), trace.c_str());
-            }
-            if (msg.rbegin()[0] == '\n') {
-                msg.pop_back();
-            }
+            msg = fmt("invalid %s: %s\n%s", describe(feature_name).c_str(), reason.c_str(), trace.c_str());
+            msg.chomp();
             errors.emplace_back(msg);
         }
         probe_stack();