[Differential] [Accepted] D4658: [KWidgetsAddons] Fix APIDOX issues raised by clang's -Wdocumentation

[Albert Astals Cid]

aacid accepted this revision.
aacid added a comment.
This revision is now accepted and ready to land.


  Looks good, fix the comments and commit!

INLINE COMMENTS

> kpagewidget.h:143
>       * This signal is emitted whenever a checkable page changes its state. @param checked is true
> -     * when the @param page is checked, or false if the @param page is unchecked.
> +     * when the @p page is checked, or false if the @param page is unchecked.
>       */

Does the other @param in this line also need to be @p ? Or even removed?

> kpagewidgetmodel.h:297
>       * This signal is emitted whenever a checkable page changes its state. @param checked is true
> -     * when the @param page is checked, or false if the @param page is unchecked.
> +     * when the @p page is checked, or false if the @param page is unchecked.
>       */

Same as above

REPOSITORY
  R236 KWidgetsAddons

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: cfeck, #frameworks, aacid
Cc: aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170218/01c6cc1d/attachment.html>