
<html><head></head><body>Maybe just add the warn_unused_result attribute?<br><br><div class="gmail_quote">On August 31, 2016 6:59:33 PM EDT, David Blaikie <dblaikie@gmail.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">dblaikie added a comment.<br /><br />I'm not as comfortable with this change - have you looked around to see if any callers are actually checking the error after calling Match?<br /><br />It seems like a client could have a problem in either case (if Regex says "no match" when it's really "ran out of memory" then maybe the code would behave incorrectly - assuming a match wasn't present when it may actually have been present).<br /><br /><br /><a href="https://reviews.llvm.org/D24102">https://reviews.llvm.org/D24102</a><br /><br /><br /><br /></pre></blockquote></div><br>

-- <br>

Sent from my Android device with K-9 Mail. Please excuse my brevity.</body></html>