On Tue, Dec 19, 2023 at 09:02:15AM -0500, Brian Foster wrote: > The have_reservation local variable in bch2_extent_fallocate() is > initialized to false and set to true further down in the function. > Between this two points, one branch of code checks for negative > value and one for positive, and nothing ever checks the variable > after it is set to true. Clean up some of the unnecessary logic and > code. > > Signed-off-by: Brian Foster <[email protected]>
This appears to have just been a leftover from when this code was extracted from the loop in fs-io.c - applied.
