[libqofono] Reiterate deprecated qt api usage. JB#59963

[pvuorela]

We don't need std::sort wrapped, it should work on all(?) Qt5 versions.

And SkipEmptyParts is available already before Qt6, let's adjust that and avoid potential warnings with new enough Qt5.

[monich]

IMO qSort(x) is far more readable than std::sort(x.begin(), x.end()) and safer because it doesn't leave a chance to write std::sort(x.begin(), y.end()) by mistake. But of course you're right, the proper C++ way is to make statements as long as possible, preferably littering them with std:: prefixes 🙂

[pvuorela]

Well, if we'd wanted it could have been a qOfonoSort wrapper that always used std::sort, but I don't think such theoretical extra safety is worth the wrapper for two simple instances.

[Tomin1]

This is fine with me.