Review Request 109965: Refactor assetimporters

Giorgos Tsiapaliokas terietor at gmail.com
Sat Apr 20 10:52:28 UTC 2013


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

(Updated April 20, 2013, 10:52 a.m.)


Review request for Plasma.


Changes
-------

update the diff according to the review


Description
-------

This patch

* removes the duplicated code in assetimporters
* adds asset's size into the db
* and fixes a few small issues


Diffs (updated)
-----

  assetimporters/CMakeLists.txt 24e76a0 
  assetimporters/database-common/channelscatalog.h 5d39c02 
  assetimporters/database-common/channelscatalog.cpp 6ca0aef 
  assetimporters/database-common/database.h 9883216 
  assetimporters/database-common/database.cpp e860bdd 
  assetimporters/kdeartwork-wallpapers/CMakeLists.txt 56d19b9 
  assetimporters/kdeartwork-wallpapers/database.h 6991758 
  assetimporters/kdeartwork-wallpapers/database.cpp d75cdda 
  assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.h PRE-CREATION 
  assetimporters/kdeartwork-wallpapers/kdewallpapersdatabase.cpp PRE-CREATION 
  assetimporters/kdeartwork-wallpapers/main.cpp 708a949 
  assetimporters/obs/CMakeLists.txt 2dbcd42 
  assetimporters/obs/channelscatalog.h PRE-CREATION 
  assetimporters/obs/channelscatalog.cpp PRE-CREATION 
  assetimporters/obs/packagedatabase.h 99f4e17 
  assetimporters/obs/packagedatabase.cpp ae43b8e 
  assetimporters/projectgutenberg/CMakeLists.txt b86cc49 
  assetimporters/projectgutenberg/src/CMakeLists.txt 2d48e9c 
  assetimporters/projectgutenberg/src/database.h 8dba0ba 
  assetimporters/projectgutenberg/src/database.cpp 75cba69 
  assetimporters/projectgutenberg/src/gutenbergdatabase.h PRE-CREATION 
  assetimporters/projectgutenberg/src/gutenbergdatabase.cpp PRE-CREATION 
  assetimporters/projectgutenberg/src/main.cpp 46f2340 
  sql/bodega.sql 44f8641 

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


Testing
-------

I haven't noticed regression.


Thanks,

Giorgos Tsiapaliokas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130420/cf830230/attachment.html>


More information about the Plasma-devel mailing list