On Thu, Jan 17, 2019 at 11:32:29PM +0800, Anand Jain wrote: > btrfs_find_device_by_devspec() finds the device by @devid or by > @device_path. This patch makes code flow easy to read by open > coding the else part and renames devpath to device_path. > > Signed-off-by: Anand Jain <[email protected]> > --- > If the else is preferred, I am ok to keep them. Nikolay?
It looks better to me the way you implement it, a series of ifs, each does a return, the function is short and easy to follow. Reviewed-by: David Sterba <[email protected]>
