Richard Henderson <richard.hender...@linaro.org> writes:
> On 4/3/20 12:11 PM, Alex Bennée wrote: >> + e->is_read = fields[1][0] == 'r' ? true : false; >> + e->is_write = fields[1][1] == 'w' ? true : false; >> + e->is_exec = fields[1][2] == 'x' ? true : false; >> + e->is_priv = fields[1][3] == 'p' ? true : false; > > Drop the redundant ? true : false. That is of course the result of the > boolean > operation. doh! Fortunately the compiler was smart enough to see through my idiocy... > >> + errors += qemu_strtoi(fields[4], NULL, 10, &e->inode); > > The root of the typedef chain for ino_t is > > /usr/include/asm-generic/posix_types.h:typedef __kernel_ulong_t > __kernel_ino_t; > > so I think you should just go ahead and use unsigned long here too. Or maybe > even uint64_t, because 32-bit has ino64_t, and could in fact have a Large > Number here. Will fix. > > > r~ -- Alex Bennée