Review Request: RSS dataengine now supports multiple feeds.

[Rob Scheepmaker]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://matt.rogers.name/r/111/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

My first contribution to the KDE project! This diff should add support for requesting a comma separated list of multiple feeds from the RSS dataengine, which will merge the entries from the different feeds in one convenient time sorted list. I'm really new to QT / plasma, so I've probably done some things wrong... so I'm looking forward to getting feedback from people who actually know what they're doing. ;)


Diffs
-----

  trunk/playground/base/plasma/engines/rss/rss.h
  trunk/playground/base/plasma/engines/rss/rss.cpp

Diff: http://matt.rogers.name/r/111/diff


Testing
-------

Not a lot atm. I've tested with both plasmaengineexplorer and the news applet in trunk. Feeding the news applet in trunk with a comma seperated list of url's shows that it works! Data abstraction rocks :). It seems however that the first time the data is requested, the feeds aren't really merged... the next update works fine. I'm still looking into that.


Thanks,

Rob