teco_string_t is now passed by value like a scalar if the callee isn't expected to modify it

* When passing a struct that should not be modified, I usually use a const pointer.
* Strings however are small 2-word objects and they are often now already passed via separate
  `gchar*` and gsize parameters. So it is consistent to pass teco_string_t by value as well.
  A teco_string_t will usually fit into registers just like a pointer.
* It's now obvious which function just _uses_ and which function _modifies_ a string.
  There is also no chance to pass a NULL pointer to those functions.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/stdio-commands.c b/src/stdio-commands.c
index bd28049..a1e3a30 100644
--- a/src/stdio-commands.c
+++ b/src/stdio-commands.c
@@ -253,9 +253,9 @@ teco_state_print_string_initial(teco_machine_main_t *ctx, GError **error)
 }
 
 static teco_state_t *
-teco_state_print_string_done(teco_machine_main_t *ctx, const teco_string_t *str, GError **error)
+teco_state_print_string_done(teco_machine_main_t *ctx, teco_string_t str, GError **error)
 {
-	teco_interface_msg_literal(TECO_MSG_USER, str->data, str->len);
+	teco_interface_msg_literal(TECO_MSG_USER, str.data, str.len);
 	return &teco_state_start;
 }