The internal api has no guarantees as to its stability.<br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 15, 2018 at 7:48 AM Greg Clayton 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">clayborg added inline comments.<br>
<br>
<br>
================<br>
Comment at: source/API/SBHostOS.cpp:48<br>
+  case ePathTypePythonDir:<br>
+    fspec = ScriptInterpreterPython::GetPythonDir();<br>
+    break;<br>
----------------<br>
Why is this here instead of inside of "HostInfo::GetLLDBPath(path_type, fspec);"? Does this mean internal code that calls:<br>
<br>
```<br>
HostInfo::GetLLDBPath(ePathTypePythonDir, fspec);<br>
```<br>
<br>
Won't work anymore? Also same for ePathTypeClangDir? Are we trying to solve a layering problems by hosing over internal clients?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D48215" rel="noreferrer" target="_blank">https://reviews.llvm.org/D48215</a><br>
<br>
<br>
<br>
</blockquote></div>