[Marble-devel] Review Request: Removing QPixmap from GeoData-classes

[Bastian Holst]

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

(Updated 2010-04-30 20:50:06.335083)


Review request for marble.


Changes
-------

Speed test.


Summary
-------

Removing QPixmap from GeoData-classes, as it is not save to be used in a thread other than the ui-thread. GeoData* are created in such a thread.


Diffs
-----

  trunk/KDE/kdeedu/marble/src/lib/VisiblePlacemark.cpp 1120603 
  trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataFeature.cpp 1120603 
  trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataIconStyle.h 1120603 
  trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataIconStyle.cpp 1120603 
  trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataStyle.h 1120603 
  trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataStyle.cpp 1120603 

Diff: http://reviewboard.kde.org/r/3855/diff


Testing (updated)
-------

Is a little slow, as GeoDataIconStyle is not implicitly shared. I'll do that soon.


Thanks,

Bastian