Review Request: Remove unneeded loop in KoDocumentRdf::emitSemanticObjectUpdated(...)

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107506/#review22781
-----------------------------------------------------------


This review has been submitted with commit 6c2e8e29d3c3cece7087b68d1a1c8dff672f6afe by Friedrich W. H. Kossebau to branch master.

- Commit Hook


On Nov. 28, 2012, 6:18 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107506/
> -----------------------------------------------------------
> 
> (Updated Nov. 28, 2012, 6:18 p.m.)
> 
> 
> 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/20121129/021ec86d/attachment.htm>