mbox series

[PATCHv4,0/3] btrfs: remove unneeded variable and if possbile

Message ID 1534437435-51605-1-git-send-email-zhongjiang@huawei.com (mailing list archive)
Headers show
Series btrfs: remove unneeded variable and if possbile | expand

Message

zhong jiang Aug. 16, 2018, 4:37 p.m. UTC
v3->v4:
 -  btrfs_destroy_delayed_refs return value would be actually needed.
    so just remove the unneeded variable "ret" and modify the subject

v2->v3:
 - repost the patches with detailed commit message.

v1->v2:
 - Fix v1 issue that forget to return after remove the variable "ret".

zhong jiang (3):
  btrfs: remove unneeded variable "ret" in btrfs_destroy_delayed_refs
  btrfs: change btrfs_free_reserved_bytes to be void funtion
  btrfs: change btrfs_pin_log_trans to be void function

 fs/btrfs/disk-io.c     | 5 ++---
 fs/btrfs/extent-tree.c | 6 ++----
 fs/btrfs/tree-log.c    | 5 +----
 fs/btrfs/tree-log.h    | 2 +-
 4 files changed, 6 insertions(+), 12 deletions(-)

Comments

David Sterba Aug. 22, 2018, 3:46 p.m. UTC | #1
On Fri, Aug 17, 2018 at 12:37:12AM +0800, zhong jiang wrote:
> v3->v4:
>  -  btrfs_destroy_delayed_refs return value would be actually needed.
>     so just remove the unneeded variable "ret" and modify the subject
> 
> v2->v3:
>  - repost the patches with detailed commit message.
> 
> v1->v2:
>  - Fix v1 issue that forget to return after remove the variable "ret".
> 
> zhong jiang (3):
>   btrfs: remove unneeded variable "ret" in btrfs_destroy_delayed_refs
>   btrfs: change btrfs_free_reserved_bytes to be void funtion
>   btrfs: change btrfs_pin_log_trans to be void function

Patches 2 and 3 applied, thanks, but the first one dropped.