<div dir="ltr">The intention in the scope of this change is just to check that the new overload is exposed correctly through the Python API.<div><br></div><div>In general, guaranteeing specific error codes (messages?) is likely impractical:</div><div><br></div><div>1. It's not always easy to do the proper checks (ex. for 'file-not-found' you'd actually get a null process, nothing more - see SBTarget::LoadCore()).</div><div>2. It's unlikely that many clients need or want to check for specific errors.</div><div>3. Such a strict contract would be very fragile (ex. adding more specific error condition detection would likely break the contract)</div><div><br></div><div>Perhaps ironically, I could take advantage of very specific error codes for my scenario, but I don't think it's feasible. So for LoadCore() I'd like to aim for more of a middle ground: accurate success/fail status + an informative error content that can be used for diagnostics (logging) and maybe as a weak hint.</div><div><br></div><div>What do you think?</div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 13, 2018 at 9:59 AM, Adrian McCarthy via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth added inline comments.<br>
<br>
<br>
================<br>
Comment at: lldb/trunk/packages/Python/<wbr>lldbsuite/test/<wbr>functionalities/postmortem/<wbr>minidump-new/TestMiniDumpNew.<wbr>py:78<br>
+        self.assertFalse(self.process, PROCESS_IS_VALID)<br>
+        self.assertTrue(error.Fail())<br>
+<br>
----------------<br>
Is it worth checking something more specific here?  That the error indicates the file was not found?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D48049" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D48049</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>