diff options
| author | Sean Wei <me@sean.taipei> | 2025-06-13 22:08:40 -0400 |
|---|---|---|
| committer | Christian Schoenebeck <qemu_oss@crudebyte.com> | 2025-07-16 14:18:48 +0200 |
| commit | 44b540338a3e271866a3d636359bfe6b2edecbb6 (patch) | |
| tree | d439deaa265768727b166eb1236752e41e498cfd | |
| parent | c079d3a31e45093286c65f8ca5350beb3a4404a9 (diff) | |
| download | qemu-44b540338a3e271866a3d636359bfe6b2edecbb6.zip qemu-44b540338a3e271866a3d636359bfe6b2edecbb6.tar.gz qemu-44b540338a3e271866a3d636359bfe6b2edecbb6.tar.bz2 | |
fsdev/9p-marshal: move G_GNUC_PRINTF to header
v9fs_string_sprintf() is annotated with G_GNUC_PRINTF(2, 3) in
9p-marshal.c, but the prototype in fsdev/9p-marshal.h is missing the
attribute, so callers that include only the header do not get format
checking.
Move the annotation to the header and delete the duplicate in the
source file. No behavior change.
Signed-off-by: Sean Wei <me@sean.taipei>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20250613.qemu.9p.01@sean.taipei>
[CS: fix code style (max. 80 chars per line)]
Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
| -rw-r--r-- | fsdev/9p-marshal.c | 3 | ||||
| -rw-r--r-- | fsdev/9p-marshal.h | 3 |
2 files changed, 3 insertions, 3 deletions
diff --git a/fsdev/9p-marshal.c b/fsdev/9p-marshal.c index f9b0336..3455580 100644 --- a/fsdev/9p-marshal.c +++ b/fsdev/9p-marshal.c @@ -27,8 +27,7 @@ void v9fs_string_free(V9fsString *str) str->size = 0; } -void G_GNUC_PRINTF(2, 3) -v9fs_string_sprintf(V9fsString *str, const char *fmt, ...) +void v9fs_string_sprintf(V9fsString *str, const char *fmt, ...) { va_list ap; diff --git a/fsdev/9p-marshal.h b/fsdev/9p-marshal.h index f1abbe1..8995e42 100644 --- a/fsdev/9p-marshal.h +++ b/fsdev/9p-marshal.h @@ -76,7 +76,8 @@ static inline void v9fs_string_init(V9fsString *str) str->size = 0; } void v9fs_string_free(V9fsString *str); -void v9fs_string_sprintf(V9fsString *str, const char *fmt, ...); +void G_GNUC_PRINTF(2, 3) v9fs_string_sprintf(V9fsString *str, const char *fmt, + ...); void v9fs_string_copy(V9fsString *lhs, V9fsString *rhs); #endif |