<div dir="ltr">Please fix some pre-existing style issues in code you moved around / refactored: * on the right, variables start with a uppercase letter, functions start with a lowercase letter. Otherwise, LGTM.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Apr 8, 2014 at 1:15 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Currently the on disk hash table's key_iterator and data_iterator make<br>
the assumption that the table data starts exactly four bytes after the<br>
base of the table. This happens to be true for all of the tables we<br>
currently iterate over, but not for all of the OnDiskHashTables we<br>
currently use. For example, key_ and data_iterator would iterate over<br>
meaningless data if they were used on the hash tables in PTHLexer.<br>
<br>
We can make the API safer by breaking this into two types. One doesn't<br>
have the iterators, and the other must be told where the payload starts.<br>
<br>
The name I've chosen, OnDiskIterableHashTable, is a bit wordy, but I<br>
couldn't think of anything better.<br>
<br>
Does this look okay to commit?<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>