Before this, the code would
return when finding empty banks with no other action (a debug message, hard to detect). Later on
m_noBanksCounter was incremented by
banks.empty() but for this line to execute, it had to be zero already! So we were not incrementing the counter (and not printing it) when the banks were empty and on the contrary we were incrementing by 0 in each event and printing the counter when banks were NOT empty.
This MR promotes
m_noBanksCounter to an
ERROR message counter and fixes the logic (such that the counter is incremented and printed when the banks are empty and not incremented neither printed when they are not).
References will need to be updated. The expected diff is the disappearing of the counters
"No bank found" from
Goes with MooreOnline!201 (merged)