删除含有bug隐患的createObject<T1>接口

该接口存在隐含的bug，相关分析写在个人博客中，如果有误，欢迎指正  https://blog.csdn.net/it_wjw/article/details/104032466

Author: Jacob0407

kbe/src/lib/common/objectpool.h

@@ -20,10 +20,10 @@ namespace KBEngine{
 #define OBJECT_POOL_INIT_SIZE			16
 #define OBJECT_POOL_INIT_MAX_SIZE		OBJECT_POOL_INIT_SIZE * 1024
 
-// 每5分钟检查一次瘦身
+// 每5分钟检查一次瘦身
 #define OBJECT_POOL_REDUCING_TIME_OUT	300 * stampsPerSecondD()
 
-// 追踪对象分配处
+// 追踪对象分配处
 #define OBJECTPOOL_POINT fmt::format("{}#{}", __FUNCTION__, __LINE__).c_str() 
 
 template< typename T >
@@ -42,9 +42,9 @@ class ObjectPoolLogPoint
 };
 
 /*
-	一些对象会非常频繁的被创建， 例如：MemoryStream, Bundle, TCPPacket等等
-	这个对象池对通过服务端峰值有效的预估提前创建出一些对象缓存起来，在用到的时候直接从对象池中
-	获取一个未被使用的对象即可。
+	一些对象会非常频繁的被创建， 例如：MemoryStream, Bundle, TCPPacket等等
+	这个对象池对通过服务端峰值有效的预估提前创建出一些对象缓存起来，在用到的时候直接从对象池中
+	获取一个未被使用的对象即可。
 */
 template< typename T, typename THREADMUTEX = KBEngine::thread::ThreadMutexNull >
 class ObjectPool
@@ -134,40 +134,8 @@ class ObjectPool
 	}
 
 	/** 
-		强制创建一个指定类型的对象。 如果缓冲里已经创建则返回现有的，否则
-		创建一个新的， 这个对象必须是继承自T的。
-	*/
-	template<typename T1>
-	T* createObject(const std::string& logPoint)
-	{
-		pMutex_->lockMutex();
-
-		while(true)
-		{
-			if(obj_count_ > 0)
-			{
-				T* t = static_cast<T1*>(*objects_.begin());
-				objects_.pop_front();
-				--obj_count_;
-				incLogPoint(logPoint);
-				t->poolObjectCreatePoint(logPoint);
-				t->onEabledPoolObject();
-				t->isEnabledPoolObject(true);
-				pMutex_->unlockMutex();
-				return t;
-			}
-
-			assignObjs();
-		}
-
-		pMutex_->unlockMutex();
-
-		return NULL;
-	}
-
-	/** 
-		创建一个对象。 如果缓冲里已经创建则返回现有的，否则
-		创建一个新的。
+		创建一个对象。 如果缓冲里已经创建则返回现有的，否则
+		创建一个新的。
 	*/
 	T* createObject(const std::string& logPoint)
 	{
@@ -197,7 +165,7 @@ class ObjectPool
 	}
 
 	/**
-		回收一个对象
+		回收一个对象
 	*/
 	void reclaimObject(T* obj)
 	{
@@ -207,7 +175,7 @@ class ObjectPool
 	}
 
 	/**
-		回收一个对象容器
+		回收一个对象容器
 	*/
 	void reclaimObject(std::list<T*>& objs)
 	{
@@ -225,7 +193,7 @@ class ObjectPool
 	}
 
 	/**
-		回收一个对象容器
+		回收一个对象容器
 	*/
 	void reclaimObject(std::vector< T* >& objs)
 	{
@@ -243,7 +211,7 @@ class ObjectPool
 	}
 
 	/**
-		回收一个对象容器
+		回收一个对象容器
 	*/
 	void reclaimObject(std::queue<T*>& objs)
 	{
@@ -296,15 +264,15 @@ class ObjectPool
 
 protected:
 	/**
-		回收一个对象
+		回收一个对象
 	*/
 	void reclaimObject_(T* obj)
 	{
 		if(obj != NULL)
 		{
 			decLogPoint(obj->poolObjectCreatePoint());
 
-			// 先重置状态
+			// 先重置状态
 			obj->onReclaimObject();
 			obj->isEnabledPoolObject(false);
 			obj->poolObjectCreatePoint("");
@@ -325,12 +293,12 @@ class ObjectPool
 
 		if (obj_count_ <= OBJECT_POOL_INIT_SIZE)
 		{
-			// 小于等于则刷新检查时间
+			// 小于等于则刷新检查时间
 			lastReducingCheckTime_ = now_timestamp;
 		}
 		else if (now_timestamp - lastReducingCheckTime_ > OBJECT_POOL_REDUCING_TIME_OUT)
 		{
-			// 长时间大于OBJECT_POOL_INIT_SIZE未使用的对象则开始做清理工作
+			// 长时间大于OBJECT_POOL_INIT_SIZE未使用的对象则开始做清理工作
 			size_t reducing = std::min(objects_.size(), std::min((size_t)OBJECT_POOL_INIT_SIZE, (size_t)(obj_count_ - OBJECT_POOL_INIT_SIZE)));
 
 			//printf("ObjectPool::reclaimObject_(): start reducing..., name=%s, currsize=%d, OBJECT_POOL_INIT_SIZE=%d\n", 
@@ -359,28 +327,28 @@ class ObjectPool
 
 	bool isDestroyed_;
 
-	// 一些原因导致锁还是有必要的
-	// 例如：dbmgr任务线程中输出log，cellapp中加载navmesh后的线程回调导致的log输出
+	// 一些原因导致锁还是有必要的
+	// 例如：dbmgr任务线程中输出log，cellapp中加载navmesh后的线程回调导致的log输出
 	THREADMUTEX* pMutex_;
 
 	std::string name_;
 
 	size_t total_allocs_;
 
-	// Linux环境中，list.size()使用的是std::distance(begin(), end())方式来获得
-	// 会对性能有影响，这里我们自己对size做一个记录
+	// Linux环境中，list.size()使用的是std::distance(begin(), end())方式来获得
+	// 会对性能有影响，这里我们自己对size做一个记录
 	size_t obj_count_;
 
-	// 最后一次瘦身检查时间
-	// 如果长达OBJECT_POOL_REDUCING_TIME_OUT大于OBJECT_POOL_INIT_SIZE，则最多瘦身OBJECT_POOL_INIT_SIZE个
+	// 最后一次瘦身检查时间
+	// 如果长达OBJECT_POOL_REDUCING_TIME_OUT大于OBJECT_POOL_INIT_SIZE，则最多瘦身OBJECT_POOL_INIT_SIZE个
 	uint64 lastReducingCheckTime_;
 
-	// 记录的创建位置信息，用于追踪泄露点
+	// 记录的创建位置信息，用于追踪泄露点
 	std::map<std::string, ObjectPoolLogPoint> logPoints_;
 };
 
 /*
-	池对象， 所有使用池的对象必须实现回收功能。
+	池对象， 所有使用池的对象必须实现回收功能。
 */
 class PoolObject
 {
@@ -402,8 +370,8 @@ class PoolObject
 	}
 
 	/**
-		池对象被析构前的通知
-		某些对象可以在此做一些工作
+		池对象被析构前的通知
+		某些对象可以在此做一些工作
 	*/
 	virtual bool destructorPoolObject()
 	{
@@ -432,10 +400,10 @@ class PoolObject
 
 protected:
 
-	// 池对象是否处于激活（从池中已经取出）状态
+	// 池对象是否处于激活（从池中已经取出）状态
 	bool isEnabledPoolObject_;
 
-	// 记录对象创建的位置
+	// 记录对象创建的位置
 	std::string poolObjectCreatePoint_;
 };