Remove useless assertions from reiserfs

Quoted 9 lines · Jeff Garzik

Why are these useless?

Quoted 5 lines

    if (len >= 12)
    {          
     //assert(len < 16);
              if (len >= 16)
                   BUG();

Seems like a valid check to me...

Just before that, there's code:

     while(len >= 16)
        {
    ...
                len -= 16;
            }

So if that ever exits with a len >=16, we have a SERIOUS problem with either the compiler or the hardware - as such, that "if (..) BUG" is dead code. Similarly for the other checks.

I always thought that assertions are just for that - if you can hit them without some unexpected event/bug, you have a SERIOUS problem.