On 12/17/20 8:21 AM, Nikolay Borisov wrote:
Instead of having 3 'if' to handle non-null return value consolidate this in 1 'if (ret)'. That way the code is more obvious:- Always drop dlete_unused_bgs_mutex if ret is non null - If ret is negative -> goto done - If it's 1 -> reset ret to 0, release the path and finish the loop. Signed-off-by: Nikolay Borisov <[email protected]>
Reviewed-by: Josef Bacik <[email protected]> Thanks, Josef
