Utils: Consolidate LayoutBuilder interface a bit

- the var args template for addItems was overkill creating a lot of
  instantiations. Use a temporary list instead.
- allow default constructed LayoutItems to be used for an empty cell,
  avoiding the use of a QLabel with empty text
- add an addRow({...}) overload as convenience shortcut for
  .startNewRow().addItems({...}
- rename startNewRow() to finishRow()

Change-Id: I6d49dacbac3d7acf140ca526884ba1ceeeca2e0d
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

1 files changed, 3 insertions, 3 deletions

diff --git a/src/plugins/incredibuild/commandbuilderaspect.cpp b/src/plugins/incredibuild/commandbuilderaspect.cpp
index 724e38d2ce..52353791a4 100644
--- a/src/plugins/incredibuild/commandbuilderaspect.cpp
+++ b/src/plugins/incredibuild/commandbuilderaspect.cpp
@@ -175,9 +175,9 @@ void CommandBuilderAspect::addToLayout(LayoutBuilder &builder)
     if (!d->m_loadedFromMap)
         d->tryToMigrate();
 
-    builder.startNewRow().addItems(d->label.data(), d->commandBuilder.data());
-    builder.startNewRow().addItems(tr("Make command:"), d->makePathChooser.data());
-    builder.startNewRow().addItems(tr("Make arguments:"), d->makeArgumentsLineEdit.data());
+    builder.addRow({d->label.data(), d->commandBuilder.data()});
+    builder.addRow({tr("Make command:"), d->makePathChooser.data()});
+    builder.addRow({tr("Make arguments:"), d->makeArgumentsLineEdit.data()});
 
     updateGui();
 }