<div dir="ltr">I'm still a bit undecided about new Error types here - I think exposing an iterator (or iterator-like thing) rather than special changes to the offset or an error kind the user checks to see whether to iterate might be better?</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 3, 2018 at 8:00 AM Jonas Devlieghere via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">JDevlieghere added a comment.<br>
<br>
I agree with @espindola, it makes more sense to have to have a separate error.<br>
<br>
In <a href="https://reviews.llvm.org/D44560#1049025" rel="noreferrer" target="_blank">https://reviews.llvm.org/D44560#1049025</a>, @jhenderson wrote:<br>
<br>
> No problem. I'm going to try experimenting with the context-based error handler a little in the meantime, and will upload another diff to present it, if I have enough time by the end of the week. FYI, I'm away on annual leave myself from Friday until Euro LLVM, so won't be reading emails until I'm back in the office (will you be around at the conference?).<br>
<br>
<br>
Yes, I look forward to meeting you!<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D44560" rel="noreferrer" target="_blank">https://reviews.llvm.org/D44560</a><br>
<br>
<br>
<br>
</blockquote></div>