dhaumann added inline comments.

INLINE COMMENTS

> filemetadataprovider.h:114
>      void loadingFinished();
> +    void dataAvailable();
>  

Could you add API documentation there what this signal indicates and when to 
use it? Typically, if you find code that is documented it is good practice to 
do the same :-)

REPOSITORY
  R824 Baloo Widgets

REVISION DETAIL
  https://phabricator.kde.org/D10113

To: michaelh, elvisangelaccio, smithjd, vhanda, ngraham, #dolphin, #frameworks
Cc: dhaumann

Reply via email to