greg/machi
1
0
Fork 0
Code Issues 11 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Bugfix: none proj must flap, undo previous commits, which may cause mess later

Browse source
This commit is contained in:
Scott Lystig Fritchie 2015-08-29 13:13:23 +09:00
parent af0ade9840
commit f21fcdd7be
1 changed files with 10 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/machi_chain_manager1.erl
View file

@ -2294,10 +2294,9 @@ calculate_flaps(P_newprop, P_latest, _P_current, CurrentUp, _FlapLimit,
{_N, _} ->
?REACT({calculate_flaps,?LINE,[]}),
false
end
andalso
%% If P_newprop is the none projection, do not start flapping.
P_newprop#projection_v1.upi /= [],
end,
%% TODO: 2015-08-29: Grr, we really need CP cases of none projection
%% flapping to propagate problem_with information.
LeaveFlapping_p =
if
LastUpChange_diff < 3.0 ->
@ -2311,10 +2310,13 @@ calculate_flaps(P_newprop, P_latest, _P_current, CurrentUp, _FlapLimit,
%% that intent.
?REACT({calculate_flaps,?LINE,[]}),
false;
AmFlappingNow_p andalso
P_newprop#projection_v1.upi == [] ->
%% P_newprop is the none projection, stop flapping.
true;
%% TODO: 2015-08-29: Grr, we really need CP cases of none projection
%% flapping to propagate problem_with information.
%% AmFlappingNow_p andalso
%% P_newprop#projection_v1.upi == [] ->
%% %% P_newprop is the none projection, stop flapping.
%% ?REACT({calculate_flaps,?LINE,[]}),
%% true;
AmFlappingNow_p andalso
CurrentUp /= FlapLastUp ->
?REACT({calculate_flaps,?LINE,[{manifesto_clause,{leave,1}}]}),