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.
--
Vincent Lefèvre <[email protected]> - Web: <http://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <http://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)
_______________________________________________
Tinycc-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/tinycc-devel
