aboutsummaryrefslogtreecommitdiff
path: root/lldb/source/Plugins/Language/CPlusPlus/LibStdcpp.cpp
diff options
context:
space:
mode:
authorRaphael Isemann <teemperor@gmail.com>2020-03-19 17:40:53 +0100
committerRaphael Isemann <teemperor@gmail.com>2020-03-19 18:10:57 +0100
commit50f198535363c6882b02c3dde7bec31e723c4762 (patch)
tree761d3bd214f896831fdb33b70d8a893d29f82639 /lldb/source/Plugins/Language/CPlusPlus/LibStdcpp.cpp
parent5ade17e0ca8b11f57cb15a1bee6d30a3815d8cac (diff)
downloadllvm-50f198535363c6882b02c3dde7bec31e723c4762.zip
llvm-50f198535363c6882b02c3dde7bec31e723c4762.tar.gz
llvm-50f198535363c6882b02c3dde7bec31e723c4762.tar.bz2
[lldb][NFC] Delete the original UserExpression before trying to reparse it with FixIts.
Currently when an expression fails to parse and we have a FixIt, we keep the failed UserExpression around while trying to parse the expression with applied fixits. This means that we have this rather confusing control flow: 1. Original expression created and parsing attempted. 2. Expression with applied FixIts is created and parsing attempted. 3. Original expression is destroyed and parser deconstructed. 4. Expression with applied FixIts is destroyed and parser deconstructed. This patch just deletes the original expression so that step 2 and 3 are swapped and the whole process looks more like just sequentially parsing two expressions (which is what we actually do here). Doesn't fix anything just makes the code less fragile.
Diffstat (limited to 'lldb/source/Plugins/Language/CPlusPlus/LibStdcpp.cpp')
0 files changed, 0 insertions, 0 deletions