diff options
author | Greg Kroah-Hartman <greg@kroah.com> | 2021-05-28 14:23:37 +0200 |
---|---|---|
committer | Kevin McCarthy <kevin@8t8.us> | 2021-05-28 09:23:27 -0700 |
commit | 88845a41681dfb3da8158d57ad3f4c8186756606 (patch) | |
tree | f87698b60db6e4ad736b49ea5dcd9a764dbeefa1 /mutt_idna.c | |
parent | 76276257bc7b82761c6cd32158c7cf00d347a236 (diff) |
fix gcc-11 build warnings for H_TO_INTL()
The macro H_TO_INTL() in mutt_idna.c, causes a lot of build warnings
about the indentation of the if () line to try to warn the developer
that it looks like this might be a bug:
mutt_idna.c: In function ‘mutt_env_to_intl’:
mutt_idna.c:327:5: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
327 | if (tag) *tag = #a; e = 1; err = NULL; \
| ^~
mutt_idna.c:333:3: note: in expansion of macro ‘H_TO_INTL’
333 | H_TO_INTL(return_path);
| ^~~~~~~~~
mutt_idna.c:327:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
327 | if (tag) *tag = #a; e = 1; err = NULL; \
| ^
mutt_idna.c:333:3: note: in expansion of macro ‘H_TO_INTL’
333 | H_TO_INTL(return_path);
| ^~~~~~~~~
This goes on for many lines.
Fix this up by properly indenting the lines so that gcc "knows" we mean
well here.
Diffstat (limited to 'mutt_idna.c')
-rw-r--r-- | mutt_idna.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/mutt_idna.c b/mutt_idna.c index db010e9b..0a37e655 100644 --- a/mutt_idna.c +++ b/mutt_idna.c @@ -324,7 +324,10 @@ void mutt_env_to_local (ENVELOPE *e) #define H_TO_INTL(a) \ if (mutt_addrlist_to_intl (env->a, err) && !e) \ { \ - if (tag) *tag = #a; e = 1; err = NULL; \ + if (tag) \ + *tag = #a; \ + e = 1; \ + err = NULL; \ } int mutt_env_to_intl (ENVELOPE *env, char **tag, char **err) |