jingham accepted this revision.
jingham added a comment.
This revision is now accepted and ready to land.
LGTM.
If the return type of the new function is not too awful to type, maybe don't
use auto? It says it's returning a path, which is usually a string, not a
FileSpec... But that's more a matter of taste.
================
Comment at: lldb/source/Core/Module.cpp:1607
std::lock_guard<std::recursive_mutex> guard(m_mutex);
- return m_source_mappings.RemapPath(path, new_path);
+ if (auto remapped = m_source_mappings.RemapPath(path)) {
+ new_path = remapped->GetPath();
----------------
RemapPath doesn't make it obvious you are getting a FileSpec back. Is the
actual return something awful to type?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104404/new/
https://reviews.llvm.org/D104404
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
