[ODE] Mistake in dInternalProcessIsland?

[Anton Savin]

Hi again,

I've found a following piece of code in dInternalProcessIsland:

         if (ofs[j1] < ofs[j2]) {
            int tmp = j1;
            j1 = j2;
            j2 = tmp;
          }

          // if either joint was tagged as -1 then it is an inactive (m=0)
          // joint that should not be considered
          if (j1==-1 || j2==-1) continue;

  In my old version (about 3 months old ) these two pieces were in
  reverse order: first there was test for -1 and only then ofs[jx]
  checked. And it's correct, of course, or I'm missing something?

  Cheers,
  Anton Savin