broulik added a comment.

  Not sure if evil or genius ;)
  Looks sane, as sane as it gets, obviously.

INLINE COMMENTS

> avahi_listener_p.h:29
> +
> +// Assits with listening to Avahi for all signals and then checking if the
> +// a given dbus message is meant for us or not.

*Assists

> avahi_listener_p.h:41
> +    // This method gets called a lot but doesn't do much. Suggest inlining.
> +    inline bool isOurMsg(const QDBusMessage &msg)
> +    {

Make `const`

REPOSITORY
  R272 KDNSSD

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

To: sitter, mdawson, broulik
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to