exception::what returns a StringView rather than a const char*

src/assert.cc

@@ -19,7 +19,7 @@ struct assert_failed : logic_error
     assert_failed(String message)
         : m_message(std::move(message)) {}
 
-    const char* what() const override { return m_message.c_str(); }
+    StringView what() const override { return m_message; }
 private:
     String m_message;
 };

src/client.cc

@@ -79,7 +79,7 @@ void Client::handle_available_input(EventMode mode)
         }
         catch (Kakoune::runtime_error& error)
         {
-            context().print_status({ error.what(), get_face("Error") });
+            context().print_status({ error.what().str(), get_face("Error") });
             context().hooks().run_hook("RuntimeError", error.what(), context());
         }
         catch (Kakoune::client_removed&)

src/client_manager.cc

@@ -41,7 +41,7 @@ Client* ClientManager::create_client(std::unique_ptr<UserInterface>&& ui,
     }
     catch (Kakoune::runtime_error& error)
     {
-        client->context().print_status({ error.what(), get_face("Error") });
+        client->context().print_status({ error.what().str(), get_face("Error") });
         client->context().hooks().run_hook("RuntimeError", error.what(),
                                            client->context());
     }

src/exception.cc

@@ -7,7 +7,7 @@
 namespace Kakoune
 {
 
-const char* exception::what() const
+StringView exception::what() const
 {
     return typeid(*this).name();
 }

src/exception.hh

@@ -9,7 +9,7 @@ namespace Kakoune
 struct exception
 {
     virtual ~exception() {}
-    virtual const char* what() const;
+    virtual StringView what() const;
 };
 
 struct runtime_error : exception
@@ -17,7 +17,7 @@ struct runtime_error : exception
     runtime_error(String what)
         : m_what(std::move(what)) {}
 
-    const char* what() const override { return m_what.c_str(); }
+    StringView what() const override { return m_what.c_str(); }
 
 private:
     String m_what;

src/main.cc

@@ -307,7 +307,7 @@ int run_client(StringView session, StringView init_command)
     }
     catch (connection_failed& e)
     {
-        fputs(e.what(), stderr);
+        fputs(e.what().zstr(), stderr);
         return -1;
     }
     return 0;
@@ -448,7 +448,7 @@ int run_filter(StringView keystr, ConstArrayView<StringView> files, bool quiet)
             {
                 if (not quiet)
                     fprintf(stderr, "error while applying keys to buffer '%s': %s\n",
-                            buffer.display_name().c_str(), err.what());
+                            buffer.display_name().c_str(), (const char*)err.what().zstr());
             }
         };
 
@@ -471,7 +471,7 @@ int run_filter(StringView keystr, ConstArrayView<StringView> files, bool quiet)
     }
     catch (Kakoune::runtime_error& err)
     {
-        fprintf(stderr, "error: %s\n", err.what());
+        fprintf(stderr, "error: %s\n", (const char*)err.what().zstr());
     }
 
     buffer_manager.clear_buffer_trash();
@@ -497,7 +497,7 @@ int run_pipe(StringView session)
     }
     catch (connection_failed& e)
     {
-        fputs(e.what(), stderr);
+        fputs(e.what().zstr(), stderr);
         return -1;
     }
     return 0;
@@ -592,7 +592,7 @@ int main(int argc, char* argv[])
         printf("Error: %s\n"
                "Valid switches:\n"
                "%s",
-               error.what(),
+               (const char*)error.what().zstr(),
                generate_switches_doc(param_desc.switches).c_str());
        return -1;
     }