The 'found' variable is 0 when declared and 'found' set to 1. Using 'TRUE' statement rather than '1' makes routine be more readable. Signed-off-by: Austin Kim <austindh.kim@xxxxxxxxx> --- kernel.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel.c b/kernel.c index f4598ea..7935d91 100644 --- a/kernel.c +++ b/kernel.c @@ -10423,7 +10423,7 @@ void read_in_kernel_config(int command) { struct syment *sp; - int ii, jj, ret, end, found=0; + int ii, jj, ret, end, found = FALSE; unsigned long size, bufsz; uint64_t magic; char *pos, *ln, *buf, *head, *tail, *val, *uncomp; @@ -10488,7 +10488,7 @@ again: while (tail < (buf + (size - 1))) { if (strncmp(tail, MAGIC_END, end)==0) { - found = 1; + found = TRUE; break; } tail++; -- 2.20.1 -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/crash-utility