Should this function have a <code>m_modified = true;</code> ?