<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Please switch over the value kinds here - that way we'll get a warning<br>
for missing switch cases if we add kinds and don't update this code.<br></blockquote><div><br></div><div>Done.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Might as well use /// so doxygen picks these comments up. Applies to the<br>
new structs as well..<br></blockquote><div> </div><div><br></div><div>Done.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>Please don't do this. Documentation in a comment is clearer than dead<br>
#if'd out code. OTOH, can this be simplified with the TrailingObjects<br>
template James introduced a while ago?<br></blockquote><div><br></div><div>Removed it.</div><div><br></div><div>For a low level type related to disk layout, the simpler way is preferred.</div><div><br></div><div><br></div><div>thanks,</div><div><br></div><div>David</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
</blockquote></div><br></div></div>