Hi Andrew,

On 10/2/25 11:29 AM, Andrew Goodbody wrote:
Testing an unsigned variable to be >= 0 will always be true so remove
this redundant test.


I checked if it wouldn't rather be an issue of the wrong type being used, but i_version is:
- initialized to 0,
- set to b2->version with b2 being pL->frag list element which defines version as a u32 as well. The version member is set from jffs2 raw inode u32 version member, so we're good as well on this side.

Thus,

Reviewed-by: Quentin Schulz <[email protected]>

Thanks!
Quentin

Reply via email to