From a2262bcafa182672a383b469bff36c905803465d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Wed, 16 Oct 2019 16:21:56 +0200 Subject: core: mark unit_*_printf() functions as taking a const Unit* They should never modify the unit argument, let's make this clear. Also see 303ee60151. --- src/core/unit-printf.c | 5 ++--- src/core/unit-printf.h | 4 ++-- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/core/unit-printf.c b/src/core/unit-printf.c index ffa12d8a92..2daaaf9971 100644 --- a/src/core/unit-printf.c +++ b/src/core/unit-printf.c @@ -171,7 +171,7 @@ static int specifier_special_directory(char specifier, const void *data, const v return 0; } -int unit_name_printf(Unit *u, const char* format, char **ret) { +int unit_name_printf(const Unit *u, const char* format, char **ret) { /* * This will use the passed string as format string and replace the following specifiers (which should all be @@ -215,8 +215,7 @@ int unit_name_printf(Unit *u, const char* format, char **ret) { return specifier_printf(format, table, u, ret); } -int unit_full_printf(Unit *u, const char *format, char **ret) { - +int unit_full_printf(const Unit *u, const char *format, char **ret) { /* This is similar to unit_name_printf() but also supports unescaping. Also, adds a couple of additional codes * (which are likely not suitable for unescaped inclusion in unit names): * diff --git a/src/core/unit-printf.h b/src/core/unit-printf.h index f3dae159d5..cb75448ae5 100644 --- a/src/core/unit-printf.h +++ b/src/core/unit-printf.h @@ -3,5 +3,5 @@ #include "unit.h" -int unit_name_printf(Unit *u, const char* text, char **ret); -int unit_full_printf(Unit *u, const char *text, char **ret); +int unit_name_printf(const Unit *u, const char* text, char **ret); +int unit_full_printf(const Unit *u, const char *text, char **ret); -- cgit v1.2.1