bruns added a comment.

  I think this is ok now, but I would like to have one +1 from any other 
frameworks developer.

INLINE COMMENTS

> formatstrings.h:21
> +#ifndef KFILEMETADATA_FORMATSTRINGS_H
> +#define KFILEMETADATA_FORMATSTRINGS_H
>  

This is a private headers, and thus should have a _p infix.

REPOSITORY
  R286 KFileMetaData

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

To: astippich, broulik, bruns, mgallien, #frameworks
Cc: ngraham, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, 
spoorun, bruns, abrahams

Reply via email to