Regan Heath wrote:
> On Wed, 20 Oct 2004 14:47:53 +0200, Anders F Björklund <afb@algonet.se> 
...
>> PS. It's mostly an aesthetic issue,
>>      although loader.d was broken...
>>      (as it uses version(Linux) now)
> 
> 
> I'd prefer if it was simply case insensitive.
> 
> I know D is case sensitive, and that we like to have consistency, but, I think this is another instance where breaking from the trend is the RightThing(TM).
> 
> Regan

I don't want the version identifiers to be case insensitive, but there's no good reason why the the predefined versions shouldn't be consistent.


Mixed case (I prefer this)...
version(Windows)
version(Linux)
version(MacOSX)
version(Darwin)


or all lowercase...
version(windows)
version(linux)
version(macosx)


Some of each hurts my eyes...
version(Windows)
version(linux)

You might as well write, "Open the Door to let roger in."

-- 
Justin (a/k/a jcc7)
http://jcc_7.tripod.com/d/

On Thu, 21 Oct 2004 22:34:58 -0500, J C Calvarese <jcc7@cox.net> wrote:
> Regan Heath wrote:
>> On Wed, 20 Oct 2004 14:47:53 +0200, Anders F Björklund <afb@algonet.se>
> ...
>>> PS. It's mostly an aesthetic issue,
>>>      although loader.d was broken...
>>>      (as it uses version(Linux) now)
>>
>>
>> I'd prefer if it was simply case insensitive.
>>
>> I know D is case sensitive, and that we like to have consistency, but, I think this is another instance where breaking from the trend is the RightThing(TM).
>>
>> Regan
>
> I don't want the version identifiers to be case insensitive,

Why not?

> but there's no good reason why the the predefined versions shouldn't be consistent.

At least if they're consistent we don't have to guess what they are.


My reasoning for making them case insensitive is that it becomes less possible for a typo to cause trouble.

i.e.

version (windows)
version (WINDOWS)
version (Windows)
version (WiNdOwS)

will all work. No guessing, just spell it write (joke intentional) and it will work as intended.

Regan.

-- 
Using M2, Opera's revolutionary e-mail client: http://www.opera.com/m2/