Review Request 109965: Refactor assetimporters

Commit Hook null at kde.org
Fri May 24 06:37:13 UTC 2013


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

(Updated May 24, 2013, 6:37 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Description
-------

This patch

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


Diffs
-----

  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/20130524/ce0ef65a/attachment.html>


More information about the Plasma-devel mailing list