diff options
| author | Robin Burchell <robin.burchell@crimson.no> | 2017-05-12 13:48:50 +0200 |
|---|---|---|
| committer | Gunnar Sletta <gunnar@crimson.no> | 2017-05-15 11:08:49 +0000 |
| commit | 3d21d3e210bf597d63596f93374e16290afef81e (patch) | |
| tree | 660e2afd67dfaf4abf004f7b28772fedc27a60ce /src/benchmarkrunner.cpp | |
| parent | 18c979c3d414fcd672ebc9ffab519ae8dd72cd17 (diff) | |
benchmarkrunner: Delete on finished(), rather than dtor
We call quit(), but some tests may be executing QML for a bit longer,
not respecting the quit flag. Tear down the engine so that the quit
happens sooner. This fixes e.g. some of the animation tests running a
bit longer after measurement is over.
Change-Id: Ie05aff3d962d537c7f5d05a53f2de12522e9035d
Reviewed-by: Gunnar Sletta <gunnar@crimson.no>
Diffstat (limited to 'src/benchmarkrunner.cpp')
| -rw-r--r-- | src/benchmarkrunner.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/benchmarkrunner.cpp b/src/benchmarkrunner.cpp index e10ea19..33dd32a 100644 --- a/src/benchmarkrunner.cpp +++ b/src/benchmarkrunner.cpp @@ -44,7 +44,6 @@ BenchmarkRunner::BenchmarkRunner() BenchmarkRunner::~BenchmarkRunner() { - delete m_view; } void BenchmarkRunner::createView() @@ -112,6 +111,8 @@ void BenchmarkRunner::start() void BenchmarkRunner::finished() { + delete m_view; + m_view = 0; std::cerr << "All done..." << std::endl; qApp->quit(); } |