version: multiple conditions

[Joakim via Digitalmars-d]

On Sunday, 28 June 2015 at 13:43:39 UTC, Artur Skawina wrote:
> On 06/28/15 05:06, Joakim via Digitalmars-d wrote:
>> So you're trying to avoid writing this?
>> 
>> version(linux) version(D_LP32)
>>     version = valistIsCharPointer;
>> else version(Windows)
>>     version = valistIsCharPointer;
>> 
>> While your version is more concise, I actually think the 
>> current form is more clear.
>
> And wrong. Spot the bug.

Eh, that idiom is not commonly used so I wasn't exactly clear on 
the syntax, but you're presumably referring to the need for 
braces?

version(linux)
{   version(D_LP32) version = valistIsCharPointer; }
else version(Windows)
     version = valistIsCharPointer;

I actually find this even more clear, the best of the bunch. :)

> That's the real reason why 'version' needs to be banned from 
> the codebase. You should *never* use D's 'version'.

If you think it needs to be banned because of some non-obvious 
bracing syntax, you don't have much to stand on.