[Lldb-commits] [PATCH] D77324: [source maps] Fix remove, insert-after and replace
walter erquinigo via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Apr 2 11:56:00 PDT 2020
wallace created this revision.
wallace added reviewers: labath, clayborg.
Herald added subscribers: lldb-commits, mgrang.
Herald added a project: LLDB.
wallace updated this revision to Diff 254576.
wallace added a comment.
fix typos
I think the quarantine is reducing my IQ overall...
In this diff of mine D77186 <https://reviews.llvm.org/D77186> I introduce a bug in the replace operation, where I was failing fast by mistake.
Besides, a similar problem existed in the insert-after operation, where it was failing fast.
Finally, the remove operation was wrong, as it was not using the indices provided by the users.
I fixed those issues and added some tests account for cases with multiple elements in these requests.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D77324
Files:
lldb/source/Interpreter/OptionValuePathMappings.cpp
lldb/test/API/functionalities/source-map/TestTargetSourceMap.py
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77324.254576.patch
Type: text/x-patch
Size: 8635 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200402/cb8f1106/attachment-0001.bin>
More information about the lldb-commits
mailing list