Should this be flagged as a warning?

[bearophile]

Bernard Helyer:

>     match(ts, TokenType.Is);
>     match(ts, TokenType.OpenParen);
>     isExp.type == parseType(ts);
>
> The bug being of course, that a type is parsed and ts is 
> modified, so the expression has side effects so it's not
> flagged as a useless expression. But the comparison still
> has no effect, so should this be flagged by DMD?

The top level operation is the ==, that is a pure expression, so 
maybe dmd should warn on this.

Bye,
bearophile