D21524: Keep existing configurations' strings               

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Aug 5 11:45:07 BST 2019


apol added a comment.


  Is this only necessary for gtk2?

INLINE COMMENTS

> appearancegtk2.cpp:125
> +{
> +    QRegularExpression regExp{propertyName + "=[^\n]*($|\n)"};
>  

const?

> appearancegtk2.cpp:136
> +    if (nonStringProperties.contains(propertyName)) {
> +        newConfigString = QString{propertyName + "=" + newValue + "\n"};
> +    } else {

This constructor isn't necessary.

> appearancegtk2.cpp:138
> +    } else {
> +        newConfigString = QString{propertyName + "=\"" + newValue + "\"\n"};
> +    }

This constructor isn't necessary

> appearancegtk2.cpp:143
> +        fileContents.replace(regExp, newConfigString);
> +    } else {
> +        fileContents = QString{newConfigString + "\n"} + fileContents;

This constructor isn't necessary.

REPOSITORY
  R99 KDE Gtk Configuration Tool

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

To: gikari, apol, #plasma
Cc: ngraham, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190805/7e05a420/attachment.html>


More information about the Plasma-devel mailing list