Re: [PATCH 2/2] Remove entryLimitAllowed() TODO from RockSwapDirRr::run().

On 10/04/2011 05:21 AM, Amos Jeffries wrote:
> On 03/10/11 18:19, Dmitry Kurochkin wrote:
>>
>> RockSwapDirRr::run() calls sd->entryLimitAllowed() when sd does not
>> have a map yet, which causes entryLimitAllowed() to use zero for the
>> lower limit. But that OK so we can remove the TODO.
>> ---
>> src/fs/rock/RockSwapDir.cc | 1 -
>> 1 files changed, 0 insertions(+), 1 deletions(-)
>>
>
> +1. Both these look okay to me. Please combine on merge though.

Merged from lp:3p2-rock as trunk r11768. From high-level logging point
of view, they are now combined (with other merged changes), but it is
still possible to separate them using bzr options. These last two
patches are not closely related, IMO, so I hope keeping them separate in
bzr history is OK.

Cheers,

Alex.
Received on Thu Oct 06 2011 - 17:06:00 MDT