Re: [PATCH] Remove SwapDir::reconfigure() arguments since they are not used.

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Thu, 22 Sep 2011 15:43:53 -0600

On 09/20/2011 08:42 PM, Amos Jeffries wrote:
> On Tue, 20 Sep 2011 20:32:56 +0400, Dmitry Kurochkin wrote:
>> Hi all.
>>
>> This is a small cleanup patch discussed recently in another thread.
>> Note it depends on rock-reconfigure-t3.patch.
>>
>>
>> Before the change, SwapDir::reconfigure() took index and path
>> arguments. But none of them was actually used: neither index nor path
>> can be changed during reconfigure. And both index and path are
>> available as SwapDir members. So there is no reason to have these
>> arguments.
>>
>>
>> Regards,
>> Dmitry
>
>
> +1. Please commit asap.

Committed as trunk r11755.

Thank you,

Alex.
Received on Thu Sep 22 2011 - 21:44:41 MDT

This archive was generated by hypermail 2.2.0 : Fri Sep 23 2011 - 12:00:03 MDT