The new code in the servicesChanged() slot looks like it is doing essentially the same as the old code in the servicesListChanged() slot. I don't think there is need for this duplication. The new code looks simpler. Though the servicesChanged() slot disconnects itself so it only runs once.