[PATCH v6 14/27] pretty: add casts for decoration option pointers
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Jun 7, 2024, 06:38 UTC
- Message-ID
- <02e79d030b38de571c65f33ac0fdb08b26114972.1717740366.git.ps@pks.im>
- In-Reply-To
- <cover.1717740365.git.ps@pks.im>
The `struct decoration_options` have a prefix and suffix field which are both non-constant, but we assign a constant pointer to them. This is safe to do because we pass them to `format_decorations()`, which never modifies these pointers, and then immediately discard the structure. Add explicit casts to avoid compilation warnings with `-Wwrite-strings`.
Signed-off-by: Patrick Steinhardt <ps@pks.im> --- pretty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/pretty.c b/pretty.c index ec05db5655..1df9d635fb 100644 --- a/pretty.c +++ b/pretty.c @@ -1584,8 +1584,8 @@ static size_t format_commit_one(struct strbuf *sb, /* in UTF-8 */ case 'D': { const struct decoration_options opts = { - .prefix = "", - .suffix = "" + .prefix = (char *) "", + .suffix = (char *) "", }; format_decorations(sb, commit, c->auto_color, &opts);
-- 2.45.2.436.gcd77e87115.dirty