[Kde-pim] Review Request: Some changes to socialutils

Martin Klapetek martin.klapetek at gmail.com
Sat Nov 10 12:32:37 GMT 2012


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

(Updated Nov. 10, 2012, 12:32 p.m.)


Review request for KDEPIM, KDEPIM-Libraries and kdewin.


Changes
-------

Revert the #include <KDateTime> back to forward declaration only.


Description
-------

Thanks to bug 308763 we noticed couple issues, here's what this fixes:

 * Adds assignment operator
 * Moves the private class to separate file
 * Gets rid of PostReply class and replace it with SocialFeedItem itself (they are practically the same)
 * Fix typo/bad-copy-paste-job in _export.h file

This should hopefully fix the windows build as well.


This addresses bug 308763.
    http://bugs.kde.org/show_bug.cgi?id=308763


Diffs (updated)
-----

  akonadi/socialutils/libakonadisocialutils_export.h 106fbdc 
  akonadi/socialutils/serializer/akonadi_serializer_socialfeeditem.cpp a153d92 
  akonadi/socialutils/socialfeeditem.h 2cbabe9 
  akonadi/socialutils/socialfeeditem.cpp 6d6572b 
  akonadi/socialutils/socialfeeditem_p.h PRE-CREATION 

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


Testing
-------

Tested with facebook resource, all compiles and works.


Thanks,

Martin Klapetek

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list