Here is a series that hopefully makes the code a bit more obvious. First patch
gives a variable a more appropraite name, since it's going to be used only for
holding an extent type and not the type of a found item.
Patch 2 simply consolidated separate 'if' statements that check the retval of
the same function into a single 'if() {} else if () {}' statement. IMO this is
cleaner.
Patch 3 massages the abhorrent code that deals with btrfs_lookup_file_extent
retval. It groups everything in a coherent 'if() {} else if () {} else {}'
construct and now it's obvious under what conditions specific code is executed.
Finally, Patch 4 removes the not_found_em labelin the same function.
Nikolay Borisov (4):
btrfs: Rename found_type to extent_type
btrfs: Consolidate retval checking of core btree functions
btrfs: Refactor retval handling of btrfs_lookup_file_extent in
btrfs_get_extent
btrfs: Remove not_found_em label from btrfs_get_extent
fs/btrfs/inode.c | 67 ++++++++++++++++++++++++------------------------
1 file changed, 34 insertions(+), 33 deletions(-)
--
2.17.1
