diff --git a/GUI/Model/Device/MultiInstrumentNotifier.cpp b/GUI/Model/Device/MultiInstrumentNotifier.cpp
index 5dd5b65743a24647ebd3182cee236a907735b32b..100406df592603e72544c6a68a220a242032924c 100644
--- a/GUI/Model/Device/MultiInstrumentNotifier.cpp
+++ b/GUI/Model/Device/MultiInstrumentNotifier.cpp
@@ -19,12 +19,6 @@ MultiInstrumentNotifier::MultiInstrumentNotifier(InstrumentModel* instruments)
: m_instruments(instruments)
{
}
-
-InstrumentModel* MultiInstrumentNotifier::instrumentModel()
-{
- return m_instruments;
-}
-
void MultiInstrumentNotifier::removeInstrument(InstrumentItem* instrument)
{
m_instruments->removeInstrument(instrument);
diff --git a/GUI/Model/Device/MultiInstrumentNotifier.h b/GUI/Model/Device/MultiInstrumentNotifier.h
index bea47301be93406a9e4bb3885dcf5eb308bdb03e..3c03f1e27ece7544fb7c7f13a2ca9d92ea851165 100644
--- a/GUI/Model/Device/MultiInstrumentNotifier.h
+++ b/GUI/Model/Device/MultiInstrumentNotifier.h
@@ -43,7 +43,7 @@ public:
MultiInstrumentNotifier(InstrumentModel* instruments);
//! The list of existing instruments.
- InstrumentModel* instrumentModel();
+ InstrumentModel* instrumentModel() { return m_instruments; }
//! Add an instrument and emit the respective signal.
template <typename T> T* addInstrumentItem();
diff --git a/GUI/View/Access/DataAccessWidget.cpp b/GUI/View/Access/DataAccessWidget.cpp
index 4688a96a75133793f616ab7e15425efc00f232fe..a95dd5ad6197f6c100b1ef1ebf1bdf84bf1fb982 100644
--- a/GUI/View/Access/DataAccessWidget.cpp
+++ b/GUI/View/Access/DataAccessWidget.cpp
@@ -49,12 +49,6 @@ DatafileItem* DataAccessWidget::realItem() const
{
return dynamic_cast<DatafileItem*>(m_item);
}
-
-QObject* DataAccessWidget::jobRealBase() const
-{
- return m_item;
-}
-
Data1DItem* DataAccessWidget::realData1DItem() const
{
if (jobItem()) {
diff --git a/GUI/View/Access/DataAccessWidget.h b/GUI/View/Access/DataAccessWidget.h
index 1468f3a5753a82770486bed52934549236ead9ab..978db183d4b583738deddc5f0a9becb986e6dfff 100644
--- a/GUI/View/Access/DataAccessWidget.h
+++ b/GUI/View/Access/DataAccessWidget.h
@@ -40,7 +40,7 @@ public:
JobItem* jobItem() const;
DatafileItem* realItem() const;
- QObject* jobRealBase() const;
+ QObject* jobRealBase() const { return m_item; }
//... Access to concrete items:
diff --git a/GUI/View/Canvas/ColorMapCanvas.cpp b/GUI/View/Canvas/ColorMapCanvas.cpp
index c13a8b9b68b91b383350d8bced65b4cf56037097..0fd42664bd11582da6852e334f331be8e2437fe0 100644
--- a/GUI/View/Canvas/ColorMapCanvas.cpp
+++ b/GUI/View/Canvas/ColorMapCanvas.cpp
@@ -40,12 +40,6 @@ void ColorMapCanvas::itemToCanvas(Data2DItem* item)
{
m_colorMap->itemToMap(item);
}
-
-ColorMap* ColorMapCanvas::colorMap()
-{
- return m_colorMap;
-}
-
QCustomPlot* ColorMapCanvas::customPlot()
{
return m_colorMap->customPlot();
diff --git a/GUI/View/Canvas/ColorMapCanvas.h b/GUI/View/Canvas/ColorMapCanvas.h
index 67d9e616699689443ea9241309035f4c4c383f18..40f8b1653899623c99ddd194b503a4ca0576e136 100644
--- a/GUI/View/Canvas/ColorMapCanvas.h
+++ b/GUI/View/Canvas/ColorMapCanvas.h
@@ -33,7 +33,7 @@ public:
void itemToCanvas(Data2DItem* item);
- ColorMap* colorMap();
+ ColorMap* colorMap() { return m_colorMap; }
QCustomPlot* customPlot();
void setStatusLabelEnabled(bool flag);
diff --git a/GUI/View/Canvas/MaskEditorActions.cpp b/GUI/View/Canvas/MaskEditorActions.cpp
index 2e83f7b522d1c084b6caa85c1b84581ac8a22ef5..9319ff6fb82b601490d9941e77d4f68ac9d55343 100644
--- a/GUI/View/Canvas/MaskEditorActions.cpp
+++ b/GUI/View/Canvas/MaskEditorActions.cpp
@@ -72,12 +72,6 @@ void MaskEditorActions::setModels(MaskContainerModel* maskContainerModel,
m_maskContainerModel = maskContainerModel;
m_selectionModel = selectionModel;
}
-
-QAction* MaskEditorActions::sendToBackAction()
-{
- return m_sendToBackAction;
-}
-
QAction* MaskEditorActions::bringToFrontAction()
{
return m_bringToFrontAction;
diff --git a/GUI/View/Canvas/MaskEditorActions.h b/GUI/View/Canvas/MaskEditorActions.h
index d0242ddf4cc44d9478ba013808fbc873661029c8..1bac33b34b597455b89320f36a34f89f82190b49 100644
--- a/GUI/View/Canvas/MaskEditorActions.h
+++ b/GUI/View/Canvas/MaskEditorActions.h
@@ -36,7 +36,7 @@ public:
void setModels(MaskContainerModel* maskContainerModel, QItemSelectionModel* selectionModel);
- QAction* sendToBackAction();
+ QAction* sendToBackAction() { return m_sendToBackAction; }
QAction* bringToFrontAction();
QList<QAction*> topToolbarActions();
diff --git a/GUI/View/Canvas/MaskEditorCanvas.cpp b/GUI/View/Canvas/MaskEditorCanvas.cpp
index 586e296562dbaf3243c33c8fba87bb63b061c4a9..8de6e1b7a060072d9b5861549183696c8a9ea2e4 100644
--- a/GUI/View/Canvas/MaskEditorCanvas.cpp
+++ b/GUI/View/Canvas/MaskEditorCanvas.cpp
@@ -66,12 +66,6 @@ void MaskEditorCanvas::resetContextMEC()
m_scene->dissociateItems();
m_statusLabel->reset();
}
-
-MaskGraphicsScene* MaskEditorCanvas::getScene()
-{
- return m_scene;
-}
-
void MaskEditorCanvas::onPresentationChange(bool pixelized)
{
m_scene->clearSelection(); // important to avoid crash (unsubscribe while calling subscribers)
diff --git a/GUI/View/Canvas/MaskEditorCanvas.h b/GUI/View/Canvas/MaskEditorCanvas.h
index def57b9cfecbd7e4cb36d9a31008a5c96ab86d0d..9d4d2dccd9505318db6f2b4cfa40cc1048f18d80 100644
--- a/GUI/View/Canvas/MaskEditorCanvas.h
+++ b/GUI/View/Canvas/MaskEditorCanvas.h
@@ -37,7 +37,7 @@ public:
void resetContextMEC();
- MaskGraphicsScene* getScene();
+ MaskGraphicsScene* getScene() { return m_scene; }
signals:
void changeActivityRequest(MaskEditorFlags::Activity);
diff --git a/GUI/View/Canvas/SpecularPlotCanvas.cpp b/GUI/View/Canvas/SpecularPlotCanvas.cpp
index e8751e4699a7187395266e13dc42c4f5e648a402..654042f31ab005877609d943ed799e3c83d8bbe7 100644
--- a/GUI/View/Canvas/SpecularPlotCanvas.cpp
+++ b/GUI/View/Canvas/SpecularPlotCanvas.cpp
@@ -39,12 +39,6 @@ void SpecularPlotCanvas::setData1DItems(const QList<Data1DItem*>& data1DItems)
{
m_plot->setData1DItems(data1DItems);
}
-
-SpecularPlot* SpecularPlotCanvas::specularPlot()
-{
- return m_plot;
-}
-
QCustomPlot* SpecularPlotCanvas::customPlot()
{
return m_plot->customPlot();
diff --git a/GUI/View/Canvas/SpecularPlotCanvas.h b/GUI/View/Canvas/SpecularPlotCanvas.h
index 6836ca724abeb46194a355a3188200e346c2e934..495543aaf9a6151d8938fd707155a6ee71ec177d 100644
--- a/GUI/View/Canvas/SpecularPlotCanvas.h
+++ b/GUI/View/Canvas/SpecularPlotCanvas.h
@@ -32,7 +32,7 @@ public:
void setData1DItems(const QList<Data1DItem*>& data1DItems);
- SpecularPlot* specularPlot();
+ SpecularPlot* specularPlot() { return m_plot; }
QCustomPlot* customPlot();
void setStatusLabelEnabled(bool flag);
diff --git a/GUI/View/Dock/DockWidgetInfo.cpp b/GUI/View/Dock/DockWidgetInfo.cpp
index b9d74795a4b9364b35c4bfddf64122ca1252413a..b623066719420bd0b96cdd80b964fba551acecb9 100644
--- a/GUI/View/Dock/DockWidgetInfo.cpp
+++ b/GUI/View/Dock/DockWidgetInfo.cpp
@@ -27,12 +27,6 @@ DockWidgetInfo::DockWidgetInfo(QDockWidget* dock, QWidget* widget, Qt::DockWidge
, m_area(area)
{
}
-
-QDockWidget* DockWidgetInfo::dock()
-{
- return m_dock;
-}
-
QWidget* DockWidgetInfo::widget()
{
return m_widget;
diff --git a/GUI/View/Dock/DockWidgetInfo.h b/GUI/View/Dock/DockWidgetInfo.h
index 5f040cbd709f2665674b75f13873b30bd3084353..2322b5a4b43c32dad57f6cb20b2a2d83af14acc2 100644
--- a/GUI/View/Dock/DockWidgetInfo.h
+++ b/GUI/View/Dock/DockWidgetInfo.h
@@ -26,7 +26,7 @@ public:
DockWidgetInfo();
DockWidgetInfo(QDockWidget* dock, QWidget* widget, Qt::DockWidgetArea area);
- QDockWidget* dock();
+ QDockWidget* dock() { return m_dock; }
QWidget* widget();
Qt::DockWidgetArea area() { return m_area; }
diff --git a/GUI/View/Instrument/DistributionSelector.cpp b/GUI/View/Instrument/DistributionSelector.cpp
index 32d0dba342748b9c9144aa05628f95d649f04ce2..735a571f45c88907a59a730438d4513b2da4c8fa 100644
--- a/GUI/View/Instrument/DistributionSelector.cpp
+++ b/GUI/View/Instrument/DistributionSelector.cpp
@@ -132,12 +132,6 @@ void DistributionSelector::createMeanSpinBox(DoubleProperty& d)
createSpinBox(d);
}
}
-
-BeamDistributionItem* DistributionSelector::item() const
-{
- return m_item;
-}
-
GUI::ID::Distributions DistributionSelector::distributions() const
{
return m_distributions;
diff --git a/GUI/View/Instrument/DistributionSelector.h b/GUI/View/Instrument/DistributionSelector.h
index 0bfac960dc293b90e8747f6ef68ff7d722621d23..d27a3cb963f0376e0efa9175d8b540cb57031565 100644
--- a/GUI/View/Instrument/DistributionSelector.h
+++ b/GUI/View/Instrument/DistributionSelector.h
@@ -47,7 +47,7 @@ public:
GUI::ID::Distributions distributions, QWidget* parent,
BeamDistributionItem* item, bool allow_distr);
- BeamDistributionItem* item() const;
+ BeamDistributionItem* item() const { return m_item; }
GUI::ID::Distributions distributions() const;
void refresh();
diff --git a/GUI/View/Main/MainWindow.cpp b/GUI/View/Main/MainWindow.cpp
index da479aa7fe1b47c61a342b2f79fbc16afb6a193d..d7c864c44f1db6bb5ee5253adbf95bc4eb287c24 100644
--- a/GUI/View/Main/MainWindow.cpp
+++ b/GUI/View/Main/MainWindow.cpp
@@ -102,12 +102,6 @@ MainWindow::MainWindow()
}
MainWindow::~MainWindow() = default;
-
-QProgressBar* MainWindow::progressBar()
-{
- return m_progressBar;
-}
-
ProjectManager* MainWindow::projectManager()
{
return m_projectManager;
diff --git a/GUI/View/Main/MainWindow.h b/GUI/View/Main/MainWindow.h
index 2d92c7bc48516f740f54636377d0c63412fc20ad..b6e4bd2a85ec9bf39a5d40987d7ae1f1c1431378 100644
--- a/GUI/View/Main/MainWindow.h
+++ b/GUI/View/Main/MainWindow.h
@@ -39,7 +39,7 @@ public:
explicit MainWindow();
~MainWindow() override;
- QProgressBar* progressBar();
+ QProgressBar* progressBar() { return m_progressBar; }
ProjectManager* projectManager();
diff --git a/GUI/View/Realspace/RealspacePanel.cpp b/GUI/View/Realspace/RealspacePanel.cpp
index 7b9eee9081eef4edb0e0f167fe7e074c20684556..0f9e13bed5594b69a32abf732bcd8fb2a809ea88 100644
--- a/GUI/View/Realspace/RealspacePanel.cpp
+++ b/GUI/View/Realspace/RealspacePanel.cpp
@@ -63,8 +63,3 @@ QSize RealspacePanel::sizeHint() const
{
return {300, 300};
}
-
-RealspaceWidget* RealspacePanel::widget()
-{
- return m_widget;
-}
diff --git a/GUI/View/Realspace/RealspacePanel.h b/GUI/View/Realspace/RealspacePanel.h
index 545540334d8ff677879af1d07f9cde9890d768b3..0af3e12a8c39366936f6dcd92e6630318c4df17a 100644
--- a/GUI/View/Realspace/RealspacePanel.h
+++ b/GUI/View/Realspace/RealspacePanel.h
@@ -27,7 +27,7 @@ public:
RealspacePanel(QWidget* parent);
QSize sizeHint() const override;
- RealspaceWidget* widget();
+ RealspaceWidget* widget() { return m_widget; }
private:
RealspaceWidget* m_widget;
diff --git a/GUI/View/Sample/CompoundForm.cpp b/GUI/View/Sample/CompoundForm.cpp
index 45051782b81bfd6ff8b765b91f767474d6baef56..cf86dcf154d4044f9884185d9c27e0bcbd921b3b 100644
--- a/GUI/View/Sample/CompoundForm.cpp
+++ b/GUI/View/Sample/CompoundForm.cpp
@@ -70,12 +70,6 @@ CompoundForm::CompoundForm(QWidget* parent, CompoundItem* compoundItem, SampleEd
}
CompoundForm::~CompoundForm() = default;
-
-CompoundItem* CompoundForm::compositionItem() const
-{
- return m_compositionItem;
-}
-
void CompoundForm::onParticleAdded(ItemWithParticles* item)
{
int index = m_compositionItem->itemsWithParticles().indexOf(item);
diff --git a/GUI/View/Sample/CompoundForm.h b/GUI/View/Sample/CompoundForm.h
index bba78a3c8339c5b88ba02c2fce69ae6ebff16d54..4d35032981adcf8ff0a30b2b6b982e9960383c83 100644
--- a/GUI/View/Sample/CompoundForm.h
+++ b/GUI/View/Sample/CompoundForm.h
@@ -30,7 +30,7 @@ public:
bool allowRemove = true);
~CompoundForm();
- CompoundItem* compositionItem() const;
+ CompoundItem* compositionItem() const { return m_compositionItem; }
void onParticleAdded(ItemWithParticles* item);
void onAboutToRemoveParticle(ItemWithParticles* item);
diff --git a/GUI/View/Sample/CoreAndShellForm.cpp b/GUI/View/Sample/CoreAndShellForm.cpp
index 51a79e1d6b33f23e685c00d3b5ddf99d4c968220..9090c2d1d748be5c2971d8c678087773677a5d75 100644
--- a/GUI/View/Sample/CoreAndShellForm.cpp
+++ b/GUI/View/Sample/CoreAndShellForm.cpp
@@ -203,8 +203,3 @@ void CoreAndShellForm::enableStructureEditing(bool b)
m_removeAction->setVisible(b);
m_duplicateAction->setVisible(b);
}
-
-CoreAndShellItem* CoreAndShellForm::coreShellItem() const
-{
- return m_item;
-}
diff --git a/GUI/View/Sample/CoreAndShellForm.h b/GUI/View/Sample/CoreAndShellForm.h
index 5459391d5d3e8df22852d2d18e9c94aaf7c48d5d..82e768571608623a098a45e136194d1832d1e806 100644
--- a/GUI/View/Sample/CoreAndShellForm.h
+++ b/GUI/View/Sample/CoreAndShellForm.h
@@ -31,7 +31,7 @@ public:
bool allowRemove = true);
void enableStructureEditing(bool b);
- CoreAndShellItem* coreShellItem() const;
+ CoreAndShellItem* coreShellItem() const { return m_item; }
void createCoreWidgets();
void createShellWidgets();
diff --git a/GUI/View/Sample/InterferenceForm.cpp b/GUI/View/Sample/InterferenceForm.cpp
index 3f6c294660e08c8588fdc72733f9a62373c516f8..c60f9c8d4c17949443ff992b4f96ebc2e314eb73 100644
--- a/GUI/View/Sample/InterferenceForm.cpp
+++ b/GUI/View/Sample/InterferenceForm.cpp
@@ -46,12 +46,6 @@ InterferenceForm::InterferenceForm(QWidget* parent, ParticleLayoutItem* layoutIt
connect(m_interferenceTypeCombo, QOverload<int>::of(&QComboBox::currentIndexChanged),
[this](int newIndex) { m_ec->selectInterference(this, newIndex); });
}
-
-ParticleLayoutItem* InterferenceForm::layoutItem() const
-{
- return m_layoutItem;
-}
-
void InterferenceForm::onInterferenceTypeChanged()
{
while (m_layout->rowCount() > 1)
diff --git a/GUI/View/Sample/InterferenceForm.h b/GUI/View/Sample/InterferenceForm.h
index ead2a6bccb814a7a594fae15a8cb56746f3f90d6..d11f93b83ca8ca1b0b7c4f4e8a08b606d7f856ee 100644
--- a/GUI/View/Sample/InterferenceForm.h
+++ b/GUI/View/Sample/InterferenceForm.h
@@ -28,7 +28,7 @@ class InterferenceForm : public CollapsibleGroupBox {
public:
InterferenceForm(QWidget* parent, ParticleLayoutItem* layoutItem, SampleEditorController* ec);
- ParticleLayoutItem* layoutItem() const;
+ ParticleLayoutItem* layoutItem() const { return m_layoutItem; }
void onInterferenceTypeChanged();
private:
diff --git a/GUI/View/Sample/LatticeTypeSelectionForm.cpp b/GUI/View/Sample/LatticeTypeSelectionForm.cpp
index 4dc8f4f347d5acd2cf66e7e487eeaf9c5ee549da..fef6f7826d59662281fd8828e1d2d60f898f3181 100644
--- a/GUI/View/Sample/LatticeTypeSelectionForm.cpp
+++ b/GUI/View/Sample/LatticeTypeSelectionForm.cpp
@@ -67,8 +67,3 @@ void LatticeTypeSelectionForm::updateXiVisibility()
m_xiLabel->setVisible(!m_interferenceItem->xiIntegration());
m_xiLabel->buddy()->setVisible(!m_interferenceItem->xiIntegration());
}
-
-Interference2DAbstractLatticeItem* LatticeTypeSelectionForm::interferenceItem() const
-{
- return m_interferenceItem;
-}
diff --git a/GUI/View/Sample/LatticeTypeSelectionForm.h b/GUI/View/Sample/LatticeTypeSelectionForm.h
index 6a6ceea982974ce23149cf72d92ff865155dabeb..f345e02d2b6556f0ffad451a8fe1044c1ca420f2 100644
--- a/GUI/View/Sample/LatticeTypeSelectionForm.h
+++ b/GUI/View/Sample/LatticeTypeSelectionForm.h
@@ -32,7 +32,7 @@ public:
void onIntegrateOverXiChanged();
void updateXiVisibility();
- Interference2DAbstractLatticeItem* interferenceItem() const;
+ Interference2DAbstractLatticeItem* interferenceItem() const { return m_interferenceItem; }
private:
Interference2DAbstractLatticeItem* m_interferenceItem;
diff --git a/GUI/View/Sample/LayerForm.cpp b/GUI/View/Sample/LayerForm.cpp
index 71dca5ff23d0a98b75832f1c0d71468014762fdc..6908c31a44bb429860cf624f08ad365ff8d78a2e 100644
--- a/GUI/View/Sample/LayerForm.cpp
+++ b/GUI/View/Sample/LayerForm.cpp
@@ -214,8 +214,3 @@ void LayerForm::onAboutToRemoveLayout(ParticleLayoutItem* layoutItem)
m_layout->removeRow(rowInLayout);
}
-
-LayerItem* LayerForm::layerItem() const
-{
- return m_layer;
-}
diff --git a/GUI/View/Sample/LayerForm.h b/GUI/View/Sample/LayerForm.h
index ab7153644d8cf1edb9b48ebb81d62e993619672c..cfed7ba7ca0cc2fdd84bf85878dba78528bfd2b2 100644
--- a/GUI/View/Sample/LayerForm.h
+++ b/GUI/View/Sample/LayerForm.h
@@ -34,7 +34,7 @@ public:
void updateLayerPositionDependentElements();
void onLayoutAdded(ParticleLayoutItem* layoutItem);
void onAboutToRemoveLayout(ParticleLayoutItem* layoutItem);
- LayerItem* layerItem() const;
+ LayerItem* layerItem() const { return m_layer; }
void updateTitle();
void expand();
diff --git a/GUI/View/Sample/MaterialInplaceForm.cpp b/GUI/View/Sample/MaterialInplaceForm.cpp
index dd371cb86d4b6a406eb121cff2a67258d7c3dfac..6bf95c3f43f0307518de1be41aaacc77082410b0 100644
--- a/GUI/View/Sample/MaterialInplaceForm.cpp
+++ b/GUI/View/Sample/MaterialInplaceForm.cpp
@@ -41,12 +41,6 @@ MaterialInplaceForm::MaterialInplaceForm(ItemWithMaterial* item, SampleEditorCon
connect(itemWithMaterial()->materialItem(), &MaterialItem::dataChanged, this,
&MaterialInplaceForm::updateValues);
}
-
-ItemWithMaterial* MaterialInplaceForm::itemWithMaterial() const
-{
- return m_item;
-}
-
void MaterialInplaceForm::updateValues()
{
for (auto* editor : findChildren<DoubleSpinBox*>()) {
diff --git a/GUI/View/Sample/MaterialInplaceForm.h b/GUI/View/Sample/MaterialInplaceForm.h
index 93abaa0a3d62430d1de75f4f60a4005c351cff7a..98d401699e1adc8474c38684198e202f06d640ee 100644
--- a/GUI/View/Sample/MaterialInplaceForm.h
+++ b/GUI/View/Sample/MaterialInplaceForm.h
@@ -27,7 +27,7 @@ class MaterialInplaceForm : public QWidget {
public:
MaterialInplaceForm(ItemWithMaterial* item, SampleEditorController* ec);
- ItemWithMaterial* itemWithMaterial() const;
+ ItemWithMaterial* itemWithMaterial() const { return m_item; }
void updateValues();
private:
diff --git a/GUI/View/Sample/MesocrystalForm.cpp b/GUI/View/Sample/MesocrystalForm.cpp
index 5d504f6cca6ffe9bb797aacad0d8cef46d23906c..596316c884363f8fa8404cc022bd9e5891589662 100644
--- a/GUI/View/Sample/MesocrystalForm.cpp
+++ b/GUI/View/Sample/MesocrystalForm.cpp
@@ -123,8 +123,3 @@ void MesocrystalForm::createBasisWidgets()
m_layout->addRow(LayerEditorUtil::createWidgetForItemWithParticles(this, m_item->basisItem(),
false, m_ec, false));
}
-
-MesocrystalItem* MesocrystalForm::mesocrystalItem() const
-{
- return m_item;
-}
diff --git a/GUI/View/Sample/MesocrystalForm.h b/GUI/View/Sample/MesocrystalForm.h
index 729fef78e894a0452d9b52c553f05ed205434ee1..302a816cfe95dcb9ddda920c55410e173002c5f9 100644
--- a/GUI/View/Sample/MesocrystalForm.h
+++ b/GUI/View/Sample/MesocrystalForm.h
@@ -30,7 +30,7 @@ public:
MesocrystalForm(QWidget* parent, MesocrystalItem* mesocrystalItem, SampleEditorController* ec,
bool allowRemove = true);
- MesocrystalItem* mesocrystalItem() const;
+ MesocrystalItem* mesocrystalItem() const { return m_item; }
void createBasisWidgets();
private:
diff --git a/GUI/View/Sample/ParticleLayoutForm.cpp b/GUI/View/Sample/ParticleLayoutForm.cpp
index 0aad34eeda87cf856267e35d746514044e1edcf1..45e4233a54775127743c0d005899fc27ab4f9ab9 100644
--- a/GUI/View/Sample/ParticleLayoutForm.cpp
+++ b/GUI/View/Sample/ParticleLayoutForm.cpp
@@ -95,12 +95,6 @@ void ParticleLayoutForm::enableStructureEditing(bool b)
for (auto* w : m_structureEditingWidgets)
w->setVisible(b);
}
-
-ParticleLayoutItem* ParticleLayoutForm::layoutItem() const
-{
- return m_layoutItem;
-}
-
void ParticleLayoutForm::onParticleAdded(ItemWithParticles* item)
{
int index = m_layoutItem->itemsWithParticles().indexOf(item);
diff --git a/GUI/View/Sample/ParticleLayoutForm.h b/GUI/View/Sample/ParticleLayoutForm.h
index c7892355e6b10862bfa8ad9c82addae66f889115..ecfde5c8307a70558be141a7f284ff0254b812a9 100644
--- a/GUI/View/Sample/ParticleLayoutForm.h
+++ b/GUI/View/Sample/ParticleLayoutForm.h
@@ -34,7 +34,7 @@ public:
SampleEditorController* ec);
void enableStructureEditing(bool b);
- ParticleLayoutItem* layoutItem() const;
+ ParticleLayoutItem* layoutItem() const { return m_layoutItem; }
void onParticleAdded(ItemWithParticles* item);
void onAboutToRemoveParticle(ItemWithParticles* item);
diff --git a/GUI/View/Sample/SampleEditorController.cpp b/GUI/View/Sample/SampleEditorController.cpp
index 30a00cdc5aa654663213b6ff2467633bd4e3d6a9..6aa5076c20ee071a33122ed93d19fff05040942a 100644
--- a/GUI/View/Sample/SampleEditorController.cpp
+++ b/GUI/View/Sample/SampleEditorController.cpp
@@ -46,12 +46,6 @@ void SampleEditorController::setSampleForm(SampleForm* view)
{
m_sampleForm = view;
}
-
-SampleItem* SampleEditorController::sampleItem() const
-{
- return m_sampleItem;
-}
-
void SampleEditorController::addLayerItem(LayerItem* before)
{
const int i = (before != nullptr) ? m_sampleItem->layerItems().indexOf(before)
@@ -356,12 +350,6 @@ MaterialModel* SampleEditorController::materialModel() const
{
return &m_sampleItem->materialModel();
}
-
-ProjectDocument* SampleEditorController::projectDocument() const
-{
- return m_document;
-}
-
void SampleEditorController::selectMaterial(ItemWithMaterial* item,
const QString& newMaterialIdentifier)
{
diff --git a/GUI/View/Sample/SampleEditorController.h b/GUI/View/Sample/SampleEditorController.h
index 7179233790957ae9796a9e1e3780f83f0ce62cee..6e22907e20ed5f3bb1823a8c53592dffda0454b6 100644
--- a/GUI/View/Sample/SampleEditorController.h
+++ b/GUI/View/Sample/SampleEditorController.h
@@ -59,13 +59,13 @@ public:
void setSampleForm(SampleForm* view);
//! The item on which this controller operates.
- SampleItem* sampleItem() const;
+ SampleItem* sampleItem() const { return m_sampleItem; }
//! The materials of the current document
MaterialModel* materialModel() const;
//! The current document
- ProjectDocument* projectDocument() const;
+ ProjectDocument* projectDocument() const { return m_document; }
void addLayerItem(LayerItem* before);
QColor findColor(int atIndex);
diff --git a/GUI/View/Sample/SampleListing.cpp b/GUI/View/Sample/SampleListing.cpp
index ffe9943d8587e8afd2196bb7e89958f1e9387dbb..f04b470055a72a9dbd20dc7addffbf1d927d35a0 100644
--- a/GUI/View/Sample/SampleListing.cpp
+++ b/GUI/View/Sample/SampleListing.cpp
@@ -138,22 +138,10 @@ SampleItem* SampleListing::currentSampleItem()
{
return m_model->itemForIndex(currentIndex());
}
-
-QAction* SampleListing::newSampleAction()
-{
- return m_newSampleAction;
-}
-
QAction* SampleListing::importSampleAction()
{
return m_importSampleAction;
}
-
-QAction* SampleListing::chooseFromLibraryAction()
-{
- return m_chooseFromLibraryAction;
-}
-
QSize SampleListing::sizeHint() const
{
QSize s = QListView::sizeHint();
diff --git a/GUI/View/Sample/SampleListing.h b/GUI/View/Sample/SampleListing.h
index 6d6c2b14070fb7f04132cfb5cdcb51a629ab73c1..d59d7811b1774ac72a2083c7fb60cf9d7214da21 100644
--- a/GUI/View/Sample/SampleListing.h
+++ b/GUI/View/Sample/SampleListing.h
@@ -32,9 +32,9 @@ public:
void restoreSelection();
- QAction* newSampleAction();
+ QAction* newSampleAction() { return m_newSampleAction; }
QAction* importSampleAction();
- QAction* chooseFromLibraryAction();
+ QAction* chooseFromLibraryAction() { return m_chooseFromLibraryAction; }
QSize sizeHint() const override;
diff --git a/PyCore/Embed/PyObjectPtr.cpp b/PyCore/Embed/PyObjectPtr.cpp
index 789398ac426ff010e8c7dba2090af4bf08e8e1dc..b30a4237ce83caf379b3276b1eea2dc473481621 100644
--- a/PyCore/Embed/PyObjectPtr.cpp
+++ b/PyCore/Embed/PyObjectPtr.cpp
@@ -30,12 +30,6 @@ PyObjectPtr::PyObjectPtr(PyObjectPtr&& other)
: PyObjectPtr(other.release())
{
}
-
-PyObject* PyObjectPtr::get()
-{
- return m_ptr;
-}
-
PyObject* PyObjectPtr::release()
{
PyObject* pyobject_ptr{m_ptr};
diff --git a/PyCore/Embed/PyObjectPtr.h b/PyCore/Embed/PyObjectPtr.h
index 2ef7905166187edc4690ddd6ddfa2fdbd77168be..aea8fe906e19fed5d45a289751648eb717d17485 100644
--- a/PyCore/Embed/PyObjectPtr.h
+++ b/PyCore/Embed/PyObjectPtr.h
@@ -33,7 +33,7 @@ public:
PyObjectPtr& operator=(const PyObjectPtr&) = delete;
//! Returns the raw pointer to the PyObject
- PyObject* get();
+ PyObject* get() { return m_ptr; }
//! Resets the container to the initial status (does _not_ release the Python resource)
void reset();
//! Resets the container to the initial status and return the PyObject pointer