Re: Review Request 120535: attica: Add const to getter methods.

2014-10-09 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120535/ --- (Updated Oct. 9, 2014, 7:54 p.m.) Status -- This change has been mar

Re: Review Request 120535: attica: Add const to getter methods.

2014-10-09 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120535/#review68181 --- Ship it! I am pretty sure this is fine. - Albert Astals Cid

Re: Review Request 120535: attica: Add const to getter methods.

2014-10-09 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120535/ --- (Updated Oct. 9, 2014, 12:02 p.m.) Review request for KDE Frameworks and

Re: Review Request 120387: 'walletListDirty' signal should be emitted if the 'kwalletd' directory is deleted

2014-10-09 Thread Arjun AK
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120387/ --- (Updated Oct. 9, 2014, 4:30 p.m.) Status -- This change has been mar

Re: Review Request 120535: attica: Add const to getter methods.

2014-10-09 Thread Sebastian Kügler
> On Oct. 8, 2014, 9:22 p.m., Albert Astals Cid wrote: > > According to > > https://techbase.kde.org/Policies/Binary_Compatibility_Issues_With_C++ > > "changing the const/volatile qualifiers of the function" is BIC > > > > Now the thing is if we allow BIC changes in frameworks like attica or n

Re: Review Request 120539: Fix incorrect count of netwm atoms

2014-10-09 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120539/#review68114 --- Ship it! Alternatively, the count could be set at runtime (ei

Review Request 120539: Fix incorrect count of netwm atoms

2014-10-09 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120539/ --- Review request for KDE Frameworks and kwin. Bugs: 339771 https://bugs