[Digikam-devel] [Bug 183171] Decode and encode metadata once only

[Marcel Wiesweg]

http://bugs.kde.org/show_bug.cgi?id=183171





--- Comment #2 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-02-04 17:18:10 ---

> > - these classes use std::vector internally, which means copying
> > is always a deep copy? (We would have to write a wrapper for
> > Qt-style shallow copies)
> 
> Currently yes. What's the use-case that requires such a shallow copy?
> I agree that copying the containers is also not desired, the simplest
> way to avoid this is using references.

If you program Qt you get used to pass around objects (string, byte array,
image) by value and internally this is implemented very cheap and clever so
that copies are only done when data is changed. Of course references is still
cheaper, but using them as return value is very limited. So nothing strictly
requires this. And it's fine for me of course that libexiv2 uses std::vector, I
just want to know about this.
---

I would suggest to solve this for 0.11 because it is a code change that,
although not too difficult, I would like to have beta-tested.


-- 
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.