diff options
author | Kevin Ottens <kevin.ottens@kdab.com> | 2015-12-14 18:33:39 +0100 |
---|---|---|
committer | Kevin Ottens <kevin.ottens@kdab.com> | 2015-12-16 13:54:27 +0000 |
commit | edb8095b8c9a9d86f6b6abccf5d4d821470a5154 (patch) | |
tree | 368e6868ed34a166d96e3e31c36be4070c31f808 /tests | |
parent | c10370585b6228c3564a22f22ec60ee24c9fe9f1 (diff) |
Change the idiom to clear null job dependencies
The "clearNull" method is kind of surprising so make QAspectJob API more
usual. Also the "clearNull" method was only used twice so change the
idiom to go through removeDependency if that one receives a null
pointer.
Change-Id: Ib74e1054ba52754b1a19d6c3e9268fa9ff14edaf
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/auto/core/qaspectjob/tst_qaspectjob.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/auto/core/qaspectjob/tst_qaspectjob.cpp b/tests/auto/core/qaspectjob/tst_qaspectjob.cpp index 8bda72b86..6864cf0f8 100644 --- a/tests/auto/core/qaspectjob/tst_qaspectjob.cpp +++ b/tests/auto/core/qaspectjob/tst_qaspectjob.cpp @@ -103,7 +103,7 @@ private Q_SLOTS: // WHEN job2.clear(); - job1->clearNullDependencies(); + job1->removeDependency(QWeakPointer<QAspectJob>()); // THEN QCOMPARE(job1->dependencies().size(), 1); |