mbox series

[0/3] Remove uneeded variable "err"

Message ID 1533481371-21349-1-git-send-email-zhongjiang@huawei.com (mailing list archive)
Headers show
Series Remove uneeded variable "err" | expand

Message

zhong jiang Aug. 5, 2018, 3:02 p.m. UTC 
zhong jiang (3):
  fs/btrfs/disk-io: Remove unneeded variable "err"
  fs/btrfs/extent-tree: remove redudant variable "err"
  fs/btrfs/tree-log: remove the unneeded variable "err"

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

Comments

Nikolay Borisov Aug. 5, 2018, 7:14 p.m. UTC | #1 
On  5.08.2018 18:02, zhong jiang wrote:
> 
> zhong jiang (3):
>   fs/btrfs/disk-io: Remove unneeded variable "err"
>   fs/btrfs/extent-tree: remove redudant variable "err"
>   fs/btrfs/tree-log: remove the unneeded variable "err"
> 
>  fs/btrfs/disk-io.c     | 12 ++++--------
>  fs/btrfs/extent-tree.c |  6 ++----
>  fs/btrfs/tree-log.c    |  5 +----
>  fs/btrfs/tree-log.h    |  2 +-
>  4 files changed, 8 insertions(+), 17 deletions(-)
> 


NAK on the whole series on the basis of using a bulk commit message
which doesn't really describe the changes of each individual commit. If
you want those changes to eventually be merged resubmit the series with
individual reasoning for every commit.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
zhong jiang Aug. 6, 2018, 2:37 a.m. UTC | #2 
On 2018/8/6 3:14, Nikolay Borisov wrote:
>
> On  5.08.2018 18:02, zhong jiang wrote:
>> zhong jiang (3):
>>   fs/btrfs/disk-io: Remove unneeded variable "err"
>>   fs/btrfs/extent-tree: remove redudant variable "err"
>>   fs/btrfs/tree-log: remove the unneeded variable "err"
>>
>>  fs/btrfs/disk-io.c     | 12 ++++--------
>>  fs/btrfs/extent-tree.c |  6 ++----
>>  fs/btrfs/tree-log.c    |  5 +----
>>  fs/btrfs/tree-log.h    |  2 +-
>>  4 files changed, 8 insertions(+), 17 deletions(-)
>>
>
> NAK on the whole series on the basis of using a bulk commit message
> which doesn't really describe the changes of each individual commit. If
> you want those changes to eventually be merged resubmit the series with
> individual reasoning for every commit.
>
>
 It's sorry for using a bulk commit message. I will repost with a patch
 because it solves the same issue.

Thanks,
zhong jiang

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html