diff options
| author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2023-02-12 09:23:09 -0800 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2023-02-12 09:25:10 -0800 |
| commit | 25616112983cd3f8bfe5379771b5bfd63b3c3621 (patch) | |
| tree | d082410bc71f27bf5dd10b2711e54a6f779e0b75 | |
| parent | 45d8f4f20665c77dd1924b78559fb2494a77ad7a (diff) | |
| download | xorg-lib-libXpm-25616112983cd3f8bfe5379771b5bfd63b3c3621.tar.gz | |
XpmCreateDataFromXpmImage: Fix misleading indentation
CrDatFrI.c: In function ‘XpmCreateDataFromXpmImage’:
CrDatFrI.c:245:13: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
245 | if (header[l])
| ^~
In file included from CrDatFrI.c:40:
XpmI.h:80:22: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
80 | #define XpmFree(ptr) free(ptr)
| ^~~~
CrDatFrI.c:247:17: note: in expansion of macro ‘XpmFree’
247 | XpmFree(header);
| ^~~~~~~
CrDatFrI.c: In function ‘CreateColors’:
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
| -rw-r--r-- | src/CrDatFrI.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/CrDatFrI.c b/src/CrDatFrI.c index 542662f..adb4786 100644 --- a/src/CrDatFrI.c +++ b/src/CrDatFrI.c @@ -241,10 +241,11 @@ XpmCreateDataFromXpmImage( /* exit point, free only locally allocated variables */ exit: if (header) { - for (l = 0; l < header_nlines; l++) + for (l = 0; l < header_nlines; l++) { if (header[l]) XpmFree(header[l]); - XpmFree(header); + } + XpmFree(header); } return(ErrorStatus); } |