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

(Updated 2010-12-22 13:35:33.811157)


Review request for Plasma.


Changes
-------

remove space in the if()  according to kdelibs coding policy


Summary
-------

when using plasmapkg -t theme -u 
/home/neoclust/.kde4/share/apps/desktoptheme/G-Remix/metadata.desktop  ( which 
is a wrong use of the tool i agree )  plasmapkg crash.

this is because archive is set to 0 and not checked before if 
(archive->open(QIODevice::ReadOnly)) {

this patch fixes this crash


Diffs (updated)
-----

  trunk/KDE/kdelibs/plasma/packagestructure.cpp 1208544 

Diff: http://svn.reviewboard.kde.org/r/6184/diff


Testing
-------

plasmapkg -t theme -u ${the file you want}


Thanks,

Nicolas

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to