diff options
| author | Jan Kratochvil <jan.kratochvil@redhat.com> | 2019-08-20 09:24:20 +0000 |
|---|---|---|
| committer | Jan Kratochvil <jan.kratochvil@redhat.com> | 2019-08-20 09:24:20 +0000 |
| commit | f9d90bc5f690de43cbfd8bd15f6f3d3e01471615 (patch) | |
| tree | 3c62c4c7ea05d724f34236e40c08bf1526f0d5ac /lldb/source/Target/ThreadPlanStepInRange.cpp | |
| parent | 12d83b427015083f40964c706a5b9e428d9d88df (diff) | |
| download | bcm5719-llvm-f9d90bc5f690de43cbfd8bd15f6f3d3e01471615.tar.gz bcm5719-llvm-f9d90bc5f690de43cbfd8bd15f6f3d3e01471615.zip | |
[lldb] D66174 `RegularExpression` cleanup
I find as a good cleanup to drop the Compile method. As I do not find TIMTOWTDI
as an advantage and there is already constructor parameter to compile the
regex.
Differential Revision: https://reviews.llvm.org/D66392
llvm-svn: 369352
Diffstat (limited to 'lldb/source/Target/ThreadPlanStepInRange.cpp')
| -rw-r--r-- | lldb/source/Target/ThreadPlanStepInRange.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lldb/source/Target/ThreadPlanStepInRange.cpp b/lldb/source/Target/ThreadPlanStepInRange.cpp index a0549c81edc..71045cc7a99 100644 --- a/lldb/source/Target/ThreadPlanStepInRange.cpp +++ b/lldb/source/Target/ThreadPlanStepInRange.cpp @@ -313,10 +313,10 @@ bool ThreadPlanStepInRange::ShouldStop(Event *event_ptr) { void ThreadPlanStepInRange::SetAvoidRegexp(const char *name) { auto name_ref = llvm::StringRef::withNullAsEmpty(name); - if (!m_avoid_regexp_up) + if (m_avoid_regexp_up) + *m_avoid_regexp_up = RegularExpression(name_ref); + else m_avoid_regexp_up.reset(new RegularExpression(name_ref)); - - m_avoid_regexp_up->Compile(name_ref); } void ThreadPlanStepInRange::SetDefaultFlagValue(uint32_t new_value) { |
