Compilation breaks in optimized mode

[zofer1]

In file boost/random/mersenne_twister.hpp
in case assert is is defined and empty, this code fails to compile:

        if(tmp.i >= n) tmp.twist();
        if(j == num_bits - 1)
            assert((tmp.x[tmp.i] & 1) == 1);
        else
            assert((tmp.x[tmp.i] & 1) == 0);

It is a simple fix to make it more organized for the compiler:

        if(tmp.i >= n) tmp.twist();
        if(j == num_bits - 1)
        {
            assert((tmp.x[tmp.i] & 1) == 1);
        }
        else
        {
            assert((tmp.x[tmp.i] & 1) == 0);
       }

With this it compiles nicely

[mborland]

I can roll the branch condition into the assert to get rid of the branches entirely. What toolchain gave you errors? This code is at least 15 years old. The empty branches should be optimized away along with the asserts