Skip to content

workspace: Simplify outputs refresh

Aleix Pol Gonzalez requested to merge work/apol/stable-outputs into master

No need to create two QSet of outputs every time we refresh. Instead just use QList::contains, which might as well be faster. This also means that we are not listing outputs in a random order.

Signed-off-by: Falko Becker falko.becker@mbition.io

This was a problem at work because some clients were expecting to get the outputs in a certain order. Not super important as it's generally a bug in the client but I think it makes the code a bit easier to understand and potentially faster.

Merge request reports