From bd56ed5faded9a0da2f6071d1816a9308afb0187 Mon Sep 17 00:00:00 2001 From: Frank LENORMAND Date: Sat, 12 Dec 2015 12:00:52 +0300 Subject: [PATCH] Avoid instanciation of a `String` object everytime the `parse_color` function is called. --- src/face_registry.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/face_registry.cc b/src/face_registry.cc index 79781c24..cf404e63 100644 --- a/src/face_registry.cc +++ b/src/face_registry.cc @@ -9,15 +9,15 @@ namespace Kakoune static Face parse_face(StringView facedesc) { - const String invalid_face_error = "invalid face description, expected [,][+]"; + constexpr StringView invalid_face_error = "invalid face description, expected [,][+]"; auto bg_it = find(facedesc, ','); auto attr_it = find(facedesc, '+'); if (bg_it != facedesc.end() and (attr_it < bg_it or (bg_it + 1) == facedesc.end())) - throw runtime_error(invalid_face_error); + throw runtime_error(invalid_face_error.str()); if (attr_it != facedesc.end() and (attr_it + 1) == facedesc.end()) - throw runtime_error(invalid_face_error); + throw runtime_error(invalid_face_error.str()); Face res; res.fg = attr_it != facedesc.begin() ? str_to_color({facedesc.begin(), std::min(attr_it, bg_it)}) : Color::Default;