[Marble-devel] Review Request 111838: Change GeoImageGraphicsItem to use QPixmap instead of QImage internally

[René Küttner]

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

Review request for Marble.


Description
-------

GeoImageGraphicsItem stores its image as a QImage at the moment. Since GeoGraphicsItems are meant for rendering and QImage is rather slow for this purpose, I changed it to use QPixmap internally since it is optimized for rendering.


Diffs
-----

  src/lib/MarbleWidgetPopupMenu.cpp ba27f36 
  src/lib/VisiblePlacemark.cpp 42c1103 
  src/lib/geodata/data/GeoDataFeature.h 4cd0374 
  src/lib/geodata/data/GeoDataFeature.cpp 8a99a85 
  src/lib/geodata/data/GeoDataIconStyle.h 5b4644f 
  src/lib/geodata/data/GeoDataIconStyle.cpp d68d0bb 

Diff: http://git.reviewboard.kde.org/r/111838/diff/


Testing
-------

I applied the patch to master and did compile marble. I also compared the visual output of GeoImageGraphicsItems before and after the patch.


Thanks,

René Küttner

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20130801/5bf357ca/attachment-0001.html>