Review Request 119997: Add documentation to KConfig::sync()

[Martin Klapetek]

> On Srp. 31, 2014, 1:25 dop., Aleix Pol Gonzalez wrote:
> > I'm not against the change, but maybe doxygen should be a bit smarter with @reimp, no?

I guess, it'd be cool actually. On the other hand, even reimpl methods should have its own documentation as their implementation can be different from the base class.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119997/#review65564
-----------------------------------------------------------


On Srp. 29, 2014, 11:42 odp., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119997/
> -----------------------------------------------------------
> 
> (Updated Srp. 29, 2014, 11:42 odp.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kconfig
> 
> 
> Description
> -------
> 
> The docs at api.kde.org shows just "Reimplemented from superclass." for the sync() method, however the superclass, KConfigBase, is not available at the api.kde.org. So this copies the documentation from KConfigBase to KConfig so the text can actually be visible at api.kde.org
> 
> 
> Diffs
> -----
> 
>   src/core/kconfig.h d7d4b7d 
> 
> Diff: https://git.reviewboard.kde.org/r/119997/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140831/9efd59b6/attachment-0001.html>