<div dir="ltr">I'll answer the other points separately, but<br><div><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">
* the iterator_type being used in static_cast before the typedef _looks_<br>
very dangerous (even if its fine and works).<br>
 - I would place typedef up the top in their own protected area to avoid<br>
a) depending on implicit compiler behaviour and b) future developer<br>
mistakes/misunderstanding.<br></blockquote><div><br></div><div>Sure.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I dont see the first-user code that we normally require for accepting<br>
these. I know its coming with a rather big followup, just would be good<br>
if there was a simple use that could be used as exemplar.<br></blockquote><div><br></div><div>I can try to look for one but it'd be completely unrelated to the main objective of the patch.<br></div>Doing that in this patch would basically pull the whole branch in.<br></div><div class="gmail_quote">If that's fine by you, I'll look for a candidate.<br clear="all"></div><br>-- <br><div class="gmail_signature">    Francesco</div>
</div></div></div>