ovyalov closed this revision.
ovyalov added a comment.
Files:
/lldb/trunk/source/Core/Debugger.cpp
Users:
ovyalov (Author)
http://reviews.llvm.org/rL247023
http://reviews.llvm.org/D12683
___
lldb-commits mailing list
lldb-commits@lists.llvm.
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
Looks good.
http://reviews.llvm.org/D12683
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/li
tfiala added inline comments.
Comment at: source/Core/Debugger.cpp:426
@@ +425,3 @@
+for (const auto& debugger: debuggers)
+debugger->Clear();
+
ovyalov wrote:
> tfiala wrote:
> > Wouldn't Clear() be considered a mutating function? So a const debugger
ovyalov added inline comments.
Comment at: source/Core/Debugger.cpp:426
@@ +425,3 @@
+for (const auto& debugger: debuggers)
+debugger->Clear();
+
tfiala wrote:
> Wouldn't Clear() be considered a mutating function? So a const debugger ref
> seems like
tfiala added a subscriber: tfiala.
Comment at: source/Core/Debugger.cpp:426
@@ +425,3 @@
+for (const auto& debugger: debuggers)
+debugger->Clear();
+
Wouldn't Clear() be considered a mutating function? So a const debugger ref
seems like maybe it shou
ovyalov added a comment.
In http://reviews.llvm.org/D12683#241155, @zturner wrote:
> What were the symptoms of this? How'd you find it?
I was debugging port forwarding with PlatformAndroid - in particular, whether
forwarded ports are cleaned up with ~PlatformAndroidRemoteGDBServer. Turned
ou
What were the symptoms of this? How'd you find it?
On Mon, Sep 7, 2015 at 6:47 PM Oleksiy Vyalov via lldb-commits <
lldb-commits@lists.llvm.org> wrote:
> ovyalov created this revision.
> ovyalov added a reviewer: clayborg.
> ovyalov added a subscriber: lldb-commits.
>
> Python locks in memory a
ovyalov created this revision.
ovyalov added a reviewer: clayborg.
ovyalov added a subscriber: lldb-commits.
Python locks in memory a few global objects like lldb.debugger,lldb.target,...
- as a consequence, ~Debugger isn't called upon shutdown.
Calling Debugger::Clear ensures that ScriptInterpr