On Tue, Feb 18, 2020 at 08:51:24AM +1100, Dave Chinner wrote: > > if ((entry->flags & XFS_ATTR_INCOMPLETE) && > > - !(context->flags & ATTR_INCOMPLETE)) > > + !context->allow_incomplete) > > continue; /* skip incomplete entries */ > > While touching this code, can you fix the trailing comment here to > be a normal one before the if() statement, or remove it because it's > largely redundant? Yes, the comment is rather pointless so I'll remove it.
