diff options
author | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-04-11 15:33:41 +0300 |
---|---|---|
committer | Anton Kudryavtsev <a.kudryavtsev@netris.ru> | 2016-04-13 04:52:22 +0000 |
commit | 0b3e45fa0ae40e1271256a134effbcc5f3e99017 (patch) | |
tree | 82f26687132372567e1a36cbb249713f431b518a /src/corelib/io/qprocess.cpp | |
parent | c327fb79e1a50c825a945e97f2c66d07a1c6d225 (diff) |
Use takeFirst() instead of first() with removeFirst()
Reduce code size and improve readability.
Change-Id: I5ec035a39cb607f15748aaa08d73f1c1bc8e4ad8
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/corelib/io/qprocess.cpp')
-rw-r--r-- | src/corelib/io/qprocess.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/corelib/io/qprocess.cpp b/src/corelib/io/qprocess.cpp index 6f11d0892f..0a0fef4cd8 100644 --- a/src/corelib/io/qprocess.cpp +++ b/src/corelib/io/qprocess.cpp @@ -2309,8 +2309,7 @@ void QProcess::start(const QString &command, OpenMode mode) return; } - QString prog = args.first(); - args.removeFirst(); + const QString prog = args.takeFirst(); start(prog, args, mode); } @@ -2552,8 +2551,7 @@ bool QProcess::startDetached(const QString &command) if (args.isEmpty()) return false; - QString prog = args.first(); - args.removeFirst(); + const QString prog = args.takeFirst(); return QProcessPrivate::startDetached(prog, args); } |