ervin accepted this revision. ervin added inline comments. INLINE COMMENTS
> componentchooseremail.cpp:109 > + } > + if (service) { > + // avoid duplicates entry when email clients are present in > mimeapps.list's Added Associations too I really meant if (!service) + continue to avoid yet another indentation level but OK, fair enough. REPOSITORY R119 Plasma Desktop BRANCH email-config REVISION DETAIL https://phabricator.kde.org/D26565 To: meven, ngraham, ervin, #plasma, bport, crossi, dvratil Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart