Use a sorted vector rather than QSet or std::set

========== Performance data for Resolving ==========
    Old instruction count: 2324724447
    New instruction count: 2317712921
    Relative change: -1 %
    Old peak memory usage: 21411584 Bytes
    New peak memory usage: 21070552 Bytes
    Relative change: -2 %
========== Performance data for Rule Execution ==========
    Old instruction count: 4231322611
    New instruction count: 4246208129
    Relative change: +0 %
    Old peak memory usage: 19239000 Bytes
    New peak memory usage: 18181072 Bytes
    Relative change: -6 %
========== Performance data for Null Build ==========
    Old instruction count: 542161499
    New instruction count: 505511534
    Relative change: -7 %
    Old peak memory usage: 13599040 Bytes
    New peak memory usage: 11892952 Bytes
    Relative change: -13 %

Change-Id: I36b3dbd3fd2a4bc7503ee7779d31bd0217000785
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>

1 files changed, 1 insertions, 1 deletions

diff --git a/src/lib/corelib/buildgraph/executorjob.cpp b/src/lib/corelib/buildgraph/executorjob.cpp
index 90b93cf8e..d1ce1fdad 100644
--- a/src/lib/corelib/buildgraph/executorjob.cpp
+++ b/src/lib/corelib/buildgraph/executorjob.cpp
@@ -105,7 +105,7 @@ void ExecutorJob::run(Transformer *t)
     t->propertiesRequestedInCommands.clear();
     QBS_CHECK(!t->outputs.isEmpty());
     m_processCommandExecutor->setProcessEnvironment(
-                (*t->outputs.begin())->product->buildEnvironment);
+                (*t->outputs.cbegin())->product->buildEnvironment);
     m_transformer = t;
     runNextCommand();
 }