diff options
author | zhong jiang <zhongjiang@huawei.com> | 2018-08-16 12:37:15 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2018-10-15 11:23:27 -0400 |
commit | 45128b08f74131753a10a584e3e5c4496b279269 (patch) | |
tree | 55ab4d1310104c9115954a3293f8c74c868fcd32 /fs/btrfs/tree-log.h | |
parent | 556f3ca88ecb7087e7e3144e429414c60a300984 (diff) |
btrfs: change btrfs_pin_log_trans to return void
btrfs_pin_log_trans defines the variable "ret" for return value, but it
is not modified after initialization. Further, I find that none of the
callers do handles the return value, so it is safe to drop the unneeded
"ret" and make it return void.
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/tree-log.h')
-rw-r--r-- | fs/btrfs/tree-log.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/tree-log.h b/fs/btrfs/tree-log.h index 7ab9bb88a639..767765031e59 100644 --- a/fs/btrfs/tree-log.h +++ b/fs/btrfs/tree-log.h | |||
@@ -65,7 +65,7 @@ int btrfs_del_inode_ref_in_log(struct btrfs_trans_handle *trans, | |||
65 | const char *name, int name_len, | 65 | const char *name, int name_len, |
66 | struct btrfs_inode *inode, u64 dirid); | 66 | struct btrfs_inode *inode, u64 dirid); |
67 | void btrfs_end_log_trans(struct btrfs_root *root); | 67 | void btrfs_end_log_trans(struct btrfs_root *root); |
68 | int btrfs_pin_log_trans(struct btrfs_root *root); | 68 | void btrfs_pin_log_trans(struct btrfs_root *root); |
69 | void btrfs_record_unlink_dir(struct btrfs_trans_handle *trans, | 69 | void btrfs_record_unlink_dir(struct btrfs_trans_handle *trans, |
70 | struct btrfs_inode *dir, struct btrfs_inode *inode, | 70 | struct btrfs_inode *dir, struct btrfs_inode *inode, |
71 | int for_rename); | 71 | int for_rename); |