[Differential] [Changed Subscribers] D3548: Add begin/end insert/remove columns to RearrangeColumns

kfunk (Kevin Funk) noreply at phabricator.kde.org
Thu Dec 29 08:28:39 UTC 2016


kfunk added inline comments.

INLINE COMMENTS

> krearrangecolumnsproxymodel.cpp:44
> +
> +    const int cc = d_ptr->m_sourceColumns.size();
> +

Please no short hand variable names.

`cc` -> `sourceColCount`?

REPOSITORY
  R275 KItemModels

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: lepagevalleeemmanuel, #frameworks
Cc: kfunk, ltoscano
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161229/3240efa9/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list