setting name to 'Instrument' is pointless

GUI/View/FromDomain/GUIObjectBuilder.cpp

@@ -79,17 +79,13 @@ void GUI::Transform::FromDomain::populateInstrumentItems(InstrumentItems* instru
 {
     ASSERT(instrumentItems);
 
-    QString name = instrument_name.isEmpty()
-                       ? QString::fromStdString(simulation.instrument().className())
-                       : instrument_name;
-
     if (const auto* gisasSimulation = dynamic_cast<const ScatteringSimulation*>(&simulation))
-        createGISASInstrumentItem(instrumentItems, *gisasSimulation, name);
+        createGISASInstrumentItem(instrumentItems, *gisasSimulation, instrument_name);
     else if (const auto* offSpecSimulation =
                  dynamic_cast<const OffSpecularSimulation*>(&simulation))
-        createOffSpecularInstrumentItem(instrumentItems, *offSpecSimulation, name);
+        createOffSpecularInstrumentItem(instrumentItems, *offSpecSimulation, instrument_name);
     else if (const auto* spec_simulation = dynamic_cast<const SpecularSimulation*>(&simulation))
-        createSpecularInstrumentItem(instrumentItems, *spec_simulation, name);
+        createSpecularInstrumentItem(instrumentItems, *spec_simulation, instrument_name);
     else
         ASSERT(0);
 }