dinterpret.d: replace static if with version

[WalterBright]

trivial

[UplinkCoder]

LGTM!
Great!

[yebblies]

How is this not churn?

[joakim-noah]

LGTM. The only wider implication I see for this is that there are several other files using some kind of static if (LOG), so if we change all those to version (LOG) also, a simple -version=LOG passed to the build will turn them all on, whereas each of their logging is turned on on a file-by-file basis now.

May be worth thinking about changing all these to debug (LOG) instead and figuring out what names you want to use to differentiate them across modules, if that kind of separation is wanted.

[yebblies]

You're right, debug (XXX) would be more appropriate.

[WalterBright]

How is this not churn?

It's not what static if is for, the way it is used in the file is more of a C++ way of doing things than a D way.

[yebblies]

Well, you know how I feel about version. But debug at least does theoretically help with attributes later on.

[yebblies]

Auto-merge toggled on