removed unreachable debug-only type validation

[firewave]

No description provided.

[firewave]

The functionality got lost when simplifyTokens2() was removed. It is possible this is no longer needed at all and can be removed instead.

[firewave]

@chrchr-github@danmar is there still needed?

"type validation" also seems misleading but I could not think of anything better.

[chrchr-github]

@chrchr-github@danmar is there still needed?

I don't know anything about it, but don't see a reason to keep this.

[firewave]

@danmar@IOBYTE do we still need this?

The warnings in question were added in f96a88e and 989235b.

[IOBYTE]

I wrote that code 14 years ago and stopped contributing 7 years ago. It was needed back then but I have no idea if it's needed now. Does it still generate warnings in daca?

[firewave]

I wrote that code 14 years ago and stopped contributing 7 years ago.

Thanks for replying.

It was needed back then but I have no idea if it's needed now. Does it still generate warnings in daca?

No, and it hasn't for a while. I also have not seen any reference to this since I started contributing again - that is why I am asking. Looks like we can just drop it then.

[firewave]

Since it has not has not been reachable for a while (the call was removed in d204c5f) and there was not much feedback I decided to remove it.