• ChengWei (23/23) Sep 14 2011 In concurrency.d (version 2.055, line 1247 - 1250) , there's some codes

ChengWei <rivercheng gmail.com> writes:

In concurrency.d (version 2.055, line 1247 - 1250) , there's some codes
related to internal implementation of List(T).

"
 Range opSlice()
        {
            return Range( cast(Node*) &m_first );
        }
"

What does this mean? m_first itself is Node *, then &m_first is Node **,
then we cast it back to (Node *)? Does it make sense?

If we change "&m_first" to "m_first", it still is weird, because in line
1169:
"
  property bool empty() const
            {
                return !m_prev.next;
            }
"

The list is empty when m_first is null, that is m_prev is null, not
m_prev.next is null.

Is my understanding of these codes completely wrong, or it is a bug in the
code?

Thanks.