kossebau added inline comments.

INLINE COMMENTS

> CMakeLists.txt:57
> +ecm_qt_declare_logging_category(kiofilewidgets_SRCS
> +    HEADER kiofilewidgets_debug.h
> +    IDENTIFIER KIO_KFILEWIDGETS_KFILEFILTERCOMBO

Perhaps name the header matching kfilefiltercombo, to make clear this one is 
not providing the general library category.

> kfilefiltercombo.cpp:29
>  #include <QLineEdit>
> +#include <QLoggingCategory>
>  

Should not be needed, served by generated header already.

REPOSITORY
  R241 KIO

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

To: ahmadsamir, #frameworks, dfaure, meven, sitter, broulik
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

Reply via email to