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

Revision history for this message
Albert Astals Cid (aacid) 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).

Done

« Back to merge proposal