PVS-Studio warning V535 for Lugaru

float Skeleton::DoConstraints(XYZ *coords,float *scale)
{
  ...
  static int i,j,k,l,m;
  ...
  for(j=0; j<numrepeats; j++){
    ...
    for(i=0; i<num_joints; i++){
      ...
      for(j=0;j<num_joints;j++){
        if(joints[j].locked)freely=0;
      }
      ...
    }
    ...
  }
  ...
}

In general, there are a lot of fragments in Lugaru where one and the same variable is used both for inner and outer loops. But functions are huge and it's difficult to understand if there is really an error.


This suspicious code was found in Lugaru project by PVS-Studio static code analyzer.
Warning message is:
V535 The variable 'j' is being used for this loop and for the outer loop. Lugaru skeleton.cpp 392

PVS-Studio is a static analyzer for detecting bugs in the source code of applications written in C, C++, C++11, C++/CX. Site: http://www.viva64.com/en/pvs-studio/