nicolasfella added a comment.

  In D24046#542147 <https://phabricator.kde.org/D24046#542147>, @jbbgameich 
wrote:
  
  > why not
  >
  >   Q_INVOKABLE void sort(int column, Qt::SortOrder order = 
Qt::AscendingOrder) override {
  >       QSortFilterProxyModel::sort(column, order);
  >   }
  >   
  >
  > instead of adding a new function?
  
  
  mostly to get rid of the argument

REPOSITORY
  R307 KPeople

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

To: nicolasfella, apol
Cc: jbbgameich, broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, 
ngraham, bruns

Reply via email to