aprantl added inline comments.
================
Comment at: source/API/SBTarget.cpp:1467
+ }
+ const ConstString csFrom(from), csTo(to);
+ if (csFrom && csTo) {
----------------
personally I would write this as:
```
if (!csFrom)
return error.SetErrorString("<from> path is empty");
if (!csTo)
return error.SetErrorString("<to> path is empty");
Log *log(lldb_private::GetLogIfAllCategoriesSet(LIBLLDB_LOG_API));
if (log)
log->Printf("SBTarget(%p)::%s: '%s' -> '%s'",
static_cast<void *>(target_sp.get()), __FUNCTION__,
from, to);
target_sp->GetImageSearchPathList().Append(csFrom, csTo, true);
```
https://reviews.llvm.org/D49739
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
