RefCounted, RunTimeTyped : Inline constructor and destructor

Since these do nothing, or almost nothing, it seems beneficial to let the compiler know that. This gives 3-7% performance improvements in specific `Gaffer::Context` benchmarks.

Author: johnhaddon

Changes

@@ -1,6 +1,11 @@
 10.6.x.x (relative to 10.6.0.0a1)
 ========
 
+Improvements
+------------
+
+- RefCounted, RunTimeTyped : Inlined constructor and destructor.
+
 Breaking Changes
 ----------------
 

include/IECore/RefCounted.h

@@ -132,7 +132,7 @@ class IECORE_API RefCounted : private boost::noncopyable
 
 		typedef size_t RefCount;
 
-		RefCounted();
+		RefCounted() : m_numRefs( 0 ) {}
 
 		/// Add a reference to the current object
 		inline void addRef() const { m_numRefs++; };
@@ -152,7 +152,7 @@ class IECORE_API RefCounted : private boost::noncopyable
 
 	protected:
 
-		virtual ~RefCounted();
+		virtual ~RefCounted() {}
 
 	private :
 

include/IECore/RunTimeTyped.h

@@ -218,8 +218,8 @@ class IECORE_API RunTimeTyped : public RefCounted
 
 		IE_CORE_DECLAREMEMBERPTR( RunTimeTyped );
 
-		RunTimeTyped();
-		~RunTimeTyped() override;
+		RunTimeTyped() {}
+		~RunTimeTyped() override {}
 
 		//! @name Type identification functions.
 		/// These functions provide useful queries about the typing

src/IECore/RefCounted.cpp

@@ -46,14 +46,6 @@ using namespace IECore;
 RunTimeTyped::Mutex RunTimeTyped::g_baseTypeIdsMutex;
 RunTimeTyped::Mutex RunTimeTyped::g_derivedTypeIdsMutex;
 
-RunTimeTyped::RunTimeTyped()
-{
-}
-
-RunTimeTyped::~RunTimeTyped()
-{
-}
-
 TypeId RunTimeTyped::typeId() const
 {
 	return staticTypeId();