From 2a18aab80da04e3f0944ae02922ab9c0251586a3 Mon Sep 17 00:00:00 2001 From: Niels Dekker Date: Tue, 20 Sep 2022 17:04:13 +0200 Subject: [PATCH] STYLE: Make protected `Object::SubjectImplementation` members private The internal helper class `SubjectImplementation` is not designed as a base class, so in general, its members should be either `private` or `public`. --- Modules/Core/Common/src/itkObject.cxx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Modules/Core/Common/src/itkObject.cxx b/Modules/Core/Common/src/itkObject.cxx index 843aba90ce9..d494f830550 100644 --- a/Modules/Core/Common/src/itkObject.cxx +++ b/Modules/Core/Common/src/itkObject.cxx @@ -88,7 +88,7 @@ class ITKCommon_HIDDEN Object::SubjectImplementation bool m_ListModified{ false }; -protected: +private: // RAII of ListModified state to ensure exception safety struct SaveRestoreListModified { @@ -111,7 +111,6 @@ class ITKCommon_HIDDEN Object::SubjectImplementation void InvokeEventRecursion(const EventObject & event, TObject * self, std::list::reverse_iterator & i); -private: std::list m_Observers; unsigned long m_Count; };