Code review comment for lp://staging/~aacid/dee-qt/more_tests

Revision history for this message
Michal Hruby (mhr3) wrote :

> Question, do we want to leave the append/remove functions in DeeListModel or
> move them to TestDeeListModel since we only use it for the QML tests?

Definitely not for the public DeeListModel, if we want to turn the wrapper into a read-write model, there should be a good reason (not just add some tests).

review: Needs Fixing

« Back to merge proposal