Changes: only call QTrackerContactDetailData::computePredicateChains() if really...
authorFriedrich W. H. Kossebau <friedrich@openismus.com>
Thu, 17 Feb 2011 11:36:11 +0000 (12:36 +0100)
committerJens Georg <jensg@openismus.com>
Sat, 19 Feb 2011 12:21:32 +0000 (13:21 +0100)
commit1a8d335e2184fd57808ae451880d5d75e8841e15
tree4bdf2cc64238312f8cdb850495e267120fb93c01
parent12fb4903c9c7b3986de2f8bc719eccbf5b0a608c
Changes: only call QTrackerContactDetailData::computePredicateChains() if really needed

RevBy: people-review

Details:
For some details m_predicateChains, m_foreignKeyChains or m_possessedChains
are by definition empty, so with the old code QTrackerContactDetailData::computePredicateChains()
was executed on every call of QTrackerContactDetail::predicateChains/foreignKeyChains/possessedChains().
Seems to give slight improvement with ./bm_qtcontacts_trackerplugin_batchsaving --contacts=1000 --batch-size=50 --full,
on average of 10 runs 0.2 s (but then the span of the test results was 5 times bigger).
Still does spare a few cycles on all request creations, anyway.
src/dao/contactdetail.cpp
src/dao/contactdetail_p.h