[phobos] length implementation for Range

David Simcha dsimcha at gmail.com
Tue May 4 12:31:51 PDT 2010


Length should be a property.  When I filed my initial bug report, there was
no such thing as @property.  If that fixes this bug, them IMHO it's fixed.

On Tue, May 4, 2010 at 3:28 PM, Andrei Alexandrescu <andrei at erdani.com>wrote:

> I think the length should be a property.
>
> Andrei
>
>
> Masahiro Nakagawa wrote:
>
>> Hi list,
>>
>> This post is my first post at Phobos ML :)
>>
>> I have one question about hasLength.
>>
>> Current implementation from std.range:
>> -----
>> template hasLength(R)
>> {
>>      enum bool hasLength = is(typeof(R.init.length) : ulong) &&
>>          !isNarrowString!R;
>> }
>> -----
>> This implementation has following issues.
>>
>> http://d.puremagic.com/issues/show_bug.cgi?id=2873
>> http://d.puremagic.com/issues/show_bug.cgi?id=3508
>>
>> But, hasLength returns true when length method is marked as @property.
>>
>> We should fix the above issue or length method should be marked as
>> @property?
>> If latter, some ranges in phobos are incomplete.
>> _______________________________________________
>> phobos mailing list
>> phobos at puremagic.com
>> http://lists.puremagic.com/mailman/listinfo/phobos
>>
> _______________________________________________
> phobos mailing list
> phobos at puremagic.com
> http://lists.puremagic.com/mailman/listinfo/phobos
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.puremagic.com/pipermail/phobos/attachments/20100504/96009068/attachment.html>


More information about the phobos mailing list