ervin requested changes to this revision.
ervin added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> fonts.cpp:600
>
> -void KFonts::updateNeedsSave()
> -{
> - setNeedsSave(m_settings->isSaveNeeded() ||
> m_fontAASettings->needsSave());
> +bool KFonts::isSaveNeeded() const {
> + return m_fontAASettings->needsSave();
{ should be on its own line
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D26228
To: bport, #plasma, ervin
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas,
GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, ahiemstra, mart
