No point starting the condition with `m_pos &&` - you know m_pos is non-NULL, that's your loop condition.