Re: [LEDE-DEV] [PATCH 2/2] firmware-utils: mkdapimg2: Fix a few issues discovered by clang

2018-04-26 Thread John Crispin
On 23/04/18 01:22, Rosen Penev wrote: I used clang -Weverything and fixed most of the warnings displayed. this is an explanation of what you did and not what the patch does     John Signed-off-by: Rosen Penev --- tools/firmware-utils/src/mkdapimg2.c | 20 +--- 1 file cha

[LEDE-DEV] [PATCH 2/2] firmware-utils: mkdapimg2: Fix a few issues discovered by clang

2018-04-22 Thread Rosen Penev
I used clang -Weverything and fixed most of the warnings displayed. Signed-off-by: Rosen Penev --- tools/firmware-utils/src/mkdapimg2.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/tools/firmware-utils/src/mkdapimg2.c b/tools/firmware-utils/src/mkdap