
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>Hi Aaron, Ben,</div><div><br></div><div>Thank again for the review. I’ve made the changes you suggested.</div><div><br></div><div>There was a question about why I reversed the iteration over the attribute list. The answer is that the attribute list built by ParsedAttributes stores attributes in reverse order. However, this is wrong because the correct order should be maintained for serialization/deserialization and error reporting. So we have to iterate rbegin->rend. Ideally the ParsedAttributes list should be fixed to store attributes in the order they appear but this looks like a lot of work.</div><div><br></div><div>Tyler</div><div><br></div><div></div></body></html>