Avoid exposing the QModelIndex in the slot
ClosedPublic

Authored by ervin on Nov 11 2015, 12:05 AM.

Details

Summary

I didn't like it when I wrote it, the more I see it the more I hate it.
Hiding it for good and making the recursion using a proper private
method.

Will also be necessary because of upcoming include changes which mean
that QModelIndex() won't be seen from the header anymore.

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin updated this revision to Diff 1224.Nov 11 2015, 12:05 AM
ervin retitled this revision from to Avoid exposing the QModelIndex in the slot.
ervin updated this object.
ervin edited the test plan for this revision. (Show Details)
ervin added reviewers: bensi, franckarrecot.
bensi accepted this revision.Nov 12 2015, 1:17 PM
bensi edited edge metadata.
This revision is now accepted and ready to land.Nov 12 2015, 1:17 PM
This revision was automatically updated to reflect the committed changes.