Review Request: Some changes to socialutils

Martin Klapetek martin.klapetek at gmail.com
Wed Nov 7 10:47:03 UTC 2012


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

Review request for KDEPIM, KDEPIM-Libraries and kdewin.


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
-----

  akonadi/socialutils/libakonadisocialutils_export.h 106fbdc 
  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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-windows/attachments/20121107/e0a54bf7/attachment.html>


More information about the Kde-windows mailing list