[v2,00/11] part2 trivial adjustments for return variable coding style

Message

Anand Jain April 18, 2024, 7:08 a.m. UTC

This is Part 2 of the series, following v1 as below. Changes includes
optimizations on top of reorganizing the return variables in each function,
as stated in the each patch, based on the received review feedback. Thank you.

v1:
  https://lore.kernel.org/linux-btrfs/[e-mail]/

Anand Jain (11):
  btrfs: btrfs_cleanup_fs_roots handle ret variable
  btrfs: btrfs_write_marked_extents rename werr and err to ret
  btrfs: __btrfs_wait_marked_extents rename werr and err to ret
  btrfs: build_backref_tree rename err and ret to ret
  btrfs: relocate_tree_blocks reuse ret instead of err
  btrfs: btrfs_recover_relocation rename ret to ret2 and err to ret
  btrfs: quick_update_accounting drop variable err
  btrfs: btrfs_qgroup_rescan_worker rename ret to ret2 and err to ret
  btrfs: lookup_extent_data_ref code optimize return
  btrfs: btrfs_drop_snapshot optimize return variable
  btrfs: btrfs_drop_subtree optimize return variable

 fs/btrfs/disk-io.c     |  33 ++++++--------
 fs/btrfs/extent-tree.c |  93 ++++++++++++++++++-------------------
 fs/btrfs/qgroup.c      |  45 +++++++++---------
 fs/btrfs/relocation.c  | 101 +++++++++++++++++++----------------------
 fs/btrfs/transaction.c |  44 ++++++++----------
 5 files changed, 146 insertions(+), 170 deletions(-)

Comments

David Sterba April 19, 2024, 5:25 p.m. UTC | #1

On Thu, Apr 18, 2024 at 03:08:32PM +0800, Anand Jain wrote:
> This is Part 2 of the series, following v1 as below. Changes includes
> optimizations on top of reorganizing the return variables in each function,
> as stated in the each patch, based on the received review feedback. Thank you.

Most of the patches are simple renames, so please add them to for-next
as before. For the commented patches we may need another round but we're
getting closer to fixing all cases.