<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Dec 5, 2017 23:49, "Nick Coghlan" <<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>> wrote:<blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Nice updates! I like this version.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Great! :)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">My one suggestion here would be to consider a dedicated exception type<br>
like "interpreters.<wbr>SubinterpreterError", rather than re-using<br>
RuntimeError directly. That way you can put the extracted traceback on<br>
a named attribute, and retain the option of potentially adding<br>
subinterpreter awareness to the traceback module in the future.</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Yeah, I already have a deferred idea item for this. :). TBH, I was on the fence about a dedicated exception type, so you've nudged me on board. :)</div><div dir="auto"><br></div><div dir="auto">-eric</div><div dir="auto"></div></div>