romangg added inline comments. INLINE COMMENTS
> zzag wrote in textinput_interface.cpp:67 > purpose_? or just keep previous name (wlPurpose)? > > https://techbase.kde.org/Policies/Frameworks_Coding_Style#Variable_Declarations wlPurpose is also an abbreviation. ;) And in the header file it is named `purpose`. Makes more sense to me to have the name without abbreviation in the API. I'll rename the function variable though. REPOSITORY R127 KWayland REVISION DETAIL https://phabricator.kde.org/D16708 To: romangg, #kwin, #frameworks Cc: zzag, kde-frameworks-devel, michaelh, ngraham, bruns