Code review comment for lp:~marcustomlinson/unity-scopes-shell/oa_loading_bar

Revision history for this message
Paweł Stołowski (stolowski) wrote :

+ QFuture<void> future = QtConcurrent::run([&]{ service_statuses = oa_client.get_service_statuses(); });

I think oa_client could be created inside the lambda?

Also, please make the lambda return std::vector<scopes::OnlineAccountClient::ServiceStatus>, then the future will be of QFuture<std::vector<...>> type and you can get the result by future's T result() getter instead of modifying the vector by reference (the lambda needs no outer variables then I think).

review: Needs Fixing

« Back to merge proposal