Re: [PATCH 1/1] Remove unnecessary branching in free-space-tree.c

2017-05-17 Thread David Sterba
On Wed, May 17, 2017 at 03:33:45AM -0700, Sahil Kang wrote: > Both btrfs_create_free_space_tree and btrfs_clear_free_space_tree > contain: > > if (ret) > return ret; > > return 0; > > The if statement is only false when ret equals zero, and since we return > zero in such cases, we

[PATCH 1/1] Remove unnecessary branching in free-space-tree.c

2017-05-17 Thread Sahil Kang
Both btrfs_create_free_space_tree and btrfs_clear_free_space_tree contain: if (ret) return ret; return 0; The if statement is only false when ret equals zero, and since we return zero in such cases, we can safely remove the branching. Signed-off-by: Sahil Kang