Review Request: KFilePlacesModel: Separating the interface from the implementation...

nf2.email at gmail.com nf2.email at gmail.com
Sun Oct 25 02:29:17 GMT 2009


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

(Updated 2009-10-25 02:29:17.671264)


Review request for kdelibs, Peter Penz, Kevin Ottens, and David Faure.


Changes
-------

Add screenshot


Summary
-------

This is a preview patch of how I would like to allow plugging different KFilePlacesModel implementations:

* There is a new interface class "KFilePlacesModelImpl".
* KFilePlacesModel::Private is replaced by a pointer to an instance of "KFilePlacesModelImpl".
* The code from kfileplacesmodel.cpp is moved to the new KFilePlacesModelKde class, which implements this interface.


Diffs
-----

  /trunk/KDE/kdelibs/kfile/kfileplacesmodel.h 1039204 
  /trunk/KDE/kdelibs/kfile/kfileplacesmodel.cpp 1039204 
  /trunk/KDE/kdelibs/kfile/kfileplacesmodel_kde.cpp PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/kfileplacesmodel_kde_p.h PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/kfileplacesmodelimpl.h PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/kfileplacesmodelimpl.cpp PRE-CREATION 

Diff: http://reviewboard.kde.org/r/1951/diff


Testing
-------


Screenshots
-----------

screenshot filechoosers and filemanagers
  http://reviewboard.kde.org/r/1951/s/238/


Thanks,

nf2





More information about the kde-core-devel mailing list