Re: [Lldb-commits] [lldb] r324743 - [Testsuite] Remove leak tests, it's not useful anymore.

2018-02-12 Thread Davide Italiano via lldb-commits
Apologies, I clearly misunderstood. I'll add this again. On Mon, Feb 12, 2018 at 2:52 AM, Pavel Labath via lldb-commits < lldb-commits@lists.llvm.org> wrote: > I think you misunderstood the purpose the purpose of this test. The > test was there to make sure that *lldb* does not leak file

Re: [Lldb-commits] [lldb] r324743 - [Testsuite] Remove leak tests, it's not useful anymore.

2018-02-12 Thread Pavel Labath via lldb-commits
I think you misunderstood the purpose the purpose of this test. The test was there to make sure that *lldb* does not leak file descriptors into the inferior (when I wrote it a couple of years ago, we were leaking about half a dozen of them). The python version check was added there just because we

Re: [Lldb-commits] [lldb] r324743 - [Testsuite] Remove leak tests, it's not useful anymore.

2018-02-09 Thread Vedant Kumar via lldb-commits
Nice! vedant > On Feb 9, 2018, at 8:06 AM, Davide Italiano via lldb-commits > wrote: > > Author: davide > Date: Fri Feb 9 08:06:39 2018 > New Revision: 324743 > > URL: http://llvm.org/viewvc/llvm-project?rev=324743=rev > Log: > [Testsuite] Remove leak tests,

[Lldb-commits] [lldb] r324743 - [Testsuite] Remove leak tests, it's not useful anymore.

2018-02-09 Thread Davide Italiano via lldb-commits
Author: davide Date: Fri Feb 9 08:06:39 2018 New Revision: 324743 URL: http://llvm.org/viewvc/llvm-project?rev=324743=rev Log: [Testsuite] Remove leak tests, it's not useful anymore. This only worked on MacOS, which now ships a newer version of python without this bug. As such, we don't leak