Review Request: Remove unneeded loop in KoDocumentRdf::emitSemanticObjectUpdated(...)
Friedrich W. H. Kossebau
kossebau at kde.org
Wed Nov 28 18:18:42 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107506/
-----------------------------------------------------------
Review request for Calligra.
Description
-------
The old code in KoDocumentRdf::emitSemanticObjectUpdated(...) does not make any sense to me, seems to be not cleaned-up code that just does not trigger a bug, only abuses more cycles than needed:
it loops over all xmlids of the item, just to call "insertReflow(col, item);" each time. Now, KoDocumentRdf::insertReflow(...) itself does loop over all xmlids and creates a reflowItem for each, then inserts that to the table/map. Doing this the number of times of xmlids, like happening with the old code, seems useless, as each time besides the first the existing entry in the col map is just overwritten, as findExtent(xmlid) will always return the same for the same xmlid.
Okay to commit to 2.6 and master?
Diffs
-----
libs/main/rdf/KoDocumentRdf.cpp 00bc1bd
Diff: http://git.reviewboard.kde.org/r/107506/diff/
Testing
-------
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121128/32596c06/attachment.htm>
More information about the calligra-devel
mailing list