On Tue, Jul 30, 2024 at 01:15:51AM GMT, viro@xxxxxxxxxx wrote: > From: Al Viro <viro@xxxxxxxxxxxxxxxxxx> > > Currently all emptiness checks are done as fd_file(...) in boolean > context (usually something like if (!fd_file(f))...); those will be > taken care of later. > > However, there's a couple of places where we do those checks as > 'store fd_file(...) into a variable, then check if this variable is > NULL' and those are harder to spot. > > Get rid of those now. > > use fd_empty() instead of extracting file and then checking it for NULL. > > Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx> > --- Reviewed-by: Christian Brauner <brauner@xxxxxxxxxx>