Skip to content

L1: bugfix: broken if() statement

Sergey Gorbunov requested to merge se.gorbunov/cbmroot:bugBraces into master

A bugfix in L1: broken if() statement because of the missing braces.

if (fmCBMmode)
              if (tr.NHits <= 3) check = 0;
              else if (tr.NHits < 3)
                check = 0;

was interpreted as

if (fmCBMmode){
              if (tr.NHits <= 3) check = 0;
              else if (tr.NHits < 3)
                check = 0;
}

but apparently, it was meant:

if (fmCBMmode){
              if (tr.NHits <= 3) check = 0;
}
else{
 if (tr.NHits < 3)
                check = 0;
}

Weird, but the bugfix doesn't change anything in the performance..

Merge request reports