<p dir="ltr">Hi Jiewen</p>
<p dir="ltr">I cannot approve the application cache commit <a href="http://gerrit.named-data.net/1213">http://gerrit.named-data.net/1213</a> , because I cannot understand it.<br>
As I pointed out, the code is mixing data structure operations with logic.<br>
One goal of NFD and ndn-cxx is simplicity, and this code violates this goal.<br>
Please implement a simpler version.</p>
<p dir="ltr">One design is to use boost::multi_index.<br>
Specifically, take the multi_index defined in ContentStore, and add a new index that orders by data->getFullName().<br>
All lookups can then take place on this index.</p>
<p dir="ltr">Although multi_index may be slower than skip list, this design is sufficient for usage in client library, because it is expected to contain less packets than a ContentStore.<br>
The benefit besides code simplicity is that it's expected to consume less memory.</p>
<p dir="ltr">Yours, Junxiao</p>