On 2013-01-31 10:52:14 +0100, Thomas Preud'homme wrote:
Le jeudi 31 janvier 2013 02:07:36, Domingo Alvarez Duarte a écrit :
switch(op) {
case '&':
pe->v &= e2.v;
break;
case '|':
pe->v |= e2.v;
break;
default:
case '^': ///////////////////////////////////////what this case
after default mean ????????
pe->v ^= e2.v;
break;
}
Looks weird indeed but I am reluctant to change it when I don't know why it
was done this way in the first place. Since the file was commited in one go, I
can't see if this result from a mistake or if it was intentional. We can
consider changing this and the other such example right after the release.
Perhaps there was a "default:" by default the author forgot to remove.
And as it is probably just useless (if op can only be '&', '|' or '^'),
no-one noticed it.