[lkml]   [2018]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] btrfs/extent_map: Remove unneeded variable "ret"
On Sat, Aug 04, 2018 at 09:19:37PM +0800, zhong jiang wrote:
>On 2018/8/4 20:59, Lu Fengqi wrote:
>> On Sat, Aug 04, 2018 at 07:13:43PM +0800, zhong jiang wrote:
>>> The ret is not used after initalization, So just remove it.
>> So why not change these two functions to void function?
> I just keep the original interface and make the minimal change.
>of course, It is also feasible that make these two functions to void function.
>I will repost if you prefer to that change. I am ok with that.

Since the return value which always is 0 is meaningless, I would prefer
to make this change.


>zhong jiang

 \ /
  Last update: 2018-08-04 18:36    [W:0.039 / U:0.404 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site