make && ctest
Here should be m_runners = runners to be exactly same as previous. I don't see much benefit of having duplicate items.
IIUC, the original code used toSet() to remove duplicates from both "m_runners" and "runners", because QSet doesn't allow duplicate items.
But toSet() returns new container m_runners and runners are unmodified.
I don't mind reverting that bit, but it seems wasteful to me to throw newRunners away, it is sorted and has no duplicates....
QStringList has a sort() method