[Pacemaker] [Question] About the rotation of the pe-file.
Andrew Beekhof
andrew at beekhof.net
Mon Jan 16 06:36:03 CET 2012
On Mon, Jan 16, 2012 at 11:48 AM, <renayama19661014 at ybb.ne.jp> wrote:
> Hi Andrew,
> Hi Lars,
>
>> >> If you want it to be between [0, max-1],
>> >> obviously that should be
>> >> while(max > 0 && sequence >= max) {
>> >> sequence -= max;
>> >> }
>
> The rotation was carried out definitely from 0 to max-1.
>
>
>> >> Though I wonder why not simply:
>> >> if (max == 0)
>> >> return;
>> >> if (sequence > max)
>> >> sequence = 0;
>
> The rotation was carried out definitely from 0 to max.
>
>> I was probably trying to get too fancy when dealing with run-time
>> reduction of max.
>> Lets go with your way :-)
>
> I think that the application of the patch of the second simple correction is good.
Its in my private tree so far:
https://github.com/beekhof/pacemaker/commit/bfbb73c
It will make its way to clusterlabs when I merge next.
More information about the Pacemaker
mailing list