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

2014-09-15 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119997/ --- (Updated Sept. 15, 2014, 10:28 a.m.) Status -- This change has been

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

2014-09-11 Thread Matthew Dawson
> On Aug. 31, 2014, 12:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-09-11 Thread Martin Klapetek
> On Aug. 31, 2014, 6:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-09-06 Thread Matthew Dawson
> On Aug. 31, 2014, 12:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-09-01 Thread Martin Klapetek
> On Aug. 31, 2014, 6:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-09-01 Thread Aleix Pol Gonzalez
> On Aug. 31, 2014, 4:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-09-01 Thread Martin Klapetek
> On Aug. 31, 2014, 6:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-08-31 Thread Matthew Dawson
> On Aug. 31, 2014, 12:57 a.m., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-08-31 Thread 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 differ

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

2014-08-31 Thread Martin Klapetek
> On Srp. 31, 2014, 6:57 dop., Matthew Dawson wrote: > > Thanks for taking a look at this. It appears KConfigBase isn't available > > on api.kde.org as it isn't documented, as kapidox hides such classes by > > default. As KConfigBase is used outside of KConfig, I'd prefer if > > KConfigBase

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

2014-08-30 Thread Matthew Dawson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119997/#review65567 --- Thanks for taking a look at this. It appears KConfigBase isn'

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

2014-08-30 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119997/#review65564 --- I'm not against the change, but maybe doxygen should be a bit

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

2014-08-29 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119997/ --- Review request for KDE Frameworks. Repository: kconfig Description