[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [54807] trunk/blender/source: BGE: Removing the source files for the PHY interfaces since they just contained virtual destructors .

Mitchell Stokes mogurijin at gmail.com
Sun Feb 24 08:09:40 CET 2013


Revision: 54807
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=54807
Author:   moguri
Date:     2013-02-24 07:09:39 +0000 (Sun, 24 Feb 2013)
Log Message:
-----------
BGE: Removing the source files for the PHY interfaces since they just contained virtual destructors. This means we had license and doc blocks for 3 lines of code, which seemed silly. This also means that ge_phys_common no longer needs to be built as a library. I tested this with CMake and SCons using GCC; hopefully this doesn't break other systems.

Modified Paths:
--------------
    trunk/blender/source/blenderplayer/CMakeLists.txt
    trunk/blender/source/creator/CMakeLists.txt
    trunk/blender/source/gameengine/CMakeLists.txt
    trunk/blender/source/gameengine/Physics/common/PHY_IController.h
    trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.h
    trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.h
    trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsController.h
    trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsEnvironment.h
    trunk/blender/source/gameengine/Physics/common/PHY_IVehicle.h
    trunk/blender/source/gameengine/SConscript

Removed Paths:
-------------
    trunk/blender/source/gameengine/Physics/common/CMakeLists.txt
    trunk/blender/source/gameengine/Physics/common/PHY_IController.cpp
    trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.cpp
    trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.cpp
    trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsController.cpp
    trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsEnvironment.cpp
    trunk/blender/source/gameengine/Physics/common/PHY_IVehicle.cpp
    trunk/blender/source/gameengine/Physics/common/SConscript

Modified: trunk/blender/source/blenderplayer/CMakeLists.txt
===================================================================
--- trunk/blender/source/blenderplayer/CMakeLists.txt	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/blenderplayer/CMakeLists.txt	2013-02-24 07:09:39 UTC (rev 54807)
@@ -103,7 +103,6 @@
 		ge_logic_ketsji 
 		ge_phys_bullet 
 		ge_phys_dummy
-		ge_phys_common 
 		ge_logic 
 		ge_rasterizer 
 		ge_oglrasterizer 

Modified: trunk/blender/source/creator/CMakeLists.txt
===================================================================
--- trunk/blender/source/creator/CMakeLists.txt	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/creator/CMakeLists.txt	2013-02-24 07:09:39 UTC (rev 54807)
@@ -884,7 +884,6 @@
 		extern_colamd
 		ge_logic_ketsji
 		extern_recastnavigation
-		ge_phys_common
 		ge_logic
 		ge_rasterizer
 		ge_oglrasterizer

Modified: trunk/blender/source/gameengine/CMakeLists.txt
===================================================================
--- trunk/blender/source/gameengine/CMakeLists.txt	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/CMakeLists.txt	2013-02-24 07:09:39 UTC (rev 54807)
@@ -37,7 +37,6 @@
 add_subdirectory(Ketsji/KXNetwork)
 add_subdirectory(Network)
 add_subdirectory(Network/LoopBackNetwork)
-add_subdirectory(Physics/common)
 add_subdirectory(Physics/Dummy)
 add_subdirectory(Rasterizer)
 add_subdirectory(Rasterizer/RAS_OpenGLRasterizer)

Deleted: trunk/blender/source/gameengine/Physics/common/CMakeLists.txt
===================================================================
--- trunk/blender/source/gameengine/Physics/common/CMakeLists.txt	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/CMakeLists.txt	2013-02-24 07:09:39 UTC (rev 54807)
@@ -1,55 +0,0 @@
-# ***** BEGIN GPL LICENSE BLOCK *****
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software Foundation,
-# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
-#
-# The Original Code is Copyright (C) 2006, Blender Foundation
-# All rights reserved.
-#
-# The Original Code is: all of this file.
-#
-# Contributor(s): Jacques Beaurain.
-#
-# ***** END GPL LICENSE BLOCK *****
-
-set(INC
-	.
-	../Dummy
-	../../../../intern/moto/include
-)
-
-set(INC_SYS
-
-)
-
-set(SRC
-	PHY_IMotionState.cpp
-	PHY_IController.cpp
-	PHY_IPhysicsController.cpp
-	PHY_IGraphicController.cpp
-	PHY_IPhysicsEnvironment.cpp
-	PHY_IVehicle.cpp
-
-	PHY_DynamicTypes.h
-	PHY_ICharacter.h
-	PHY_IController.h
-	PHY_IGraphicController.h
-	PHY_IMotionState.h
-	PHY_IPhysicsController.h
-	PHY_IPhysicsEnvironment.h
-	PHY_IVehicle.h
-	PHY_Pro.h
-)
-
-blender_add_lib(ge_phys_common "${SRC}" "${INC}" "${INC_SYS}")

Deleted: trunk/blender/source/gameengine/Physics/common/PHY_IController.cpp
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IController.cpp	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IController.cpp	2013-02-24 07:09:39 UTC (rev 54807)
@@ -1,38 +0,0 @@
-/*
- * ***** BEGIN GPL LICENSE BLOCK *****
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * The Original Code is Copyright (C) 2001-2002 by NaN Holding BV.
- * All rights reserved.
- *
- * The Original Code is: all of this file.
- *
- * Contributor(s): none yet.
- *
- * ***** END GPL LICENSE BLOCK *****
- */
-
-/** \file gameengine/Physics/common/PHY_IController.cpp
- *  \ingroup phys
- */
-
-#include "PHY_IController.h"
-
-PHY_IController::~PHY_IController()
-{
-
-}
-

Modified: trunk/blender/source/gameengine/Physics/common/PHY_IController.h
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IController.h	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IController.h	2013-02-24 07:09:39 UTC (rev 54807)
@@ -48,7 +48,7 @@
 class PHY_IController
 {
 	public:
-		virtual ~PHY_IController();
+		virtual ~PHY_IController(){};
 		// clientinfo for raycasts for example
 		virtual	void*	getNewClientInfo()=0;
 		virtual	void	setNewClientInfo(void* clientinfo)=0;

Deleted: trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.cpp
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.cpp	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.cpp	2013-02-24 07:09:39 UTC (rev 54807)
@@ -1,38 +0,0 @@
-/*
- * ***** BEGIN GPL LICENSE BLOCK *****
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * The Original Code is Copyright (C) 2001-2002 by NaN Holding BV.
- * All rights reserved.
- *
- * The Original Code is: all of this file.
- *
- * Contributor(s): none yet.
- *
- * ***** END GPL LICENSE BLOCK *****
- */
-
-/** \file gameengine/Physics/common/PHY_IGraphicController.cpp
- *  \ingroup phys
- */
-
-#include "PHY_IGraphicController.h"
-
-PHY_IGraphicController::~PHY_IGraphicController()
-{
-
-}
-

Modified: trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.h
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.h	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IGraphicController.h	2013-02-24 07:09:39 UTC (rev 54807)
@@ -42,7 +42,6 @@
 class PHY_IGraphicController : public PHY_IController
 {
 	public:
-		virtual ~PHY_IGraphicController();
 		/**
 		 * SynchronizeMotionStates ynchronizes dynas, kinematic and deformable entities (and do 'late binding')
 		 */

Deleted: trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.cpp
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.cpp	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.cpp	2013-02-24 07:09:39 UTC (rev 54807)
@@ -1,37 +0,0 @@
-/*
- * ***** BEGIN GPL LICENSE BLOCK *****
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * The Original Code is Copyright (C) 2001-2002 by NaN Holding BV.
- * All rights reserved.
- *
- * The Original Code is: all of this file.
- *
- * Contributor(s): none yet.
- *
- * ***** END GPL LICENSE BLOCK *****
- */
-
-/** \file gameengine/Physics/common/PHY_IMotionState.cpp
- *  \ingroup phys
- */
-
-#include "PHY_IMotionState.h"
-
-PHY_IMotionState::~PHY_IMotionState()
-{
-
-}

Modified: trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.h
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.h	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IMotionState.h	2013-02-24 07:09:39 UTC (rev 54807)
@@ -44,7 +44,7 @@
 
 {
 	public:
-		virtual ~PHY_IMotionState();
+		virtual ~PHY_IMotionState(){};
 
 		virtual void	getWorldPosition(float& posX,float& posY,float& posZ)=0;
 		virtual void	getWorldScaling(float& scaleX,float& scaleY,float& scaleZ)=0;

Deleted: trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsController.cpp
===================================================================
--- trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsController.cpp	2013-02-24 05:05:29 UTC (rev 54806)
+++ trunk/blender/source/gameengine/Physics/common/PHY_IPhysicsController.cpp	2013-02-24 07:09:39 UTC (rev 54807)
@@ -1,38 +0,0 @@
-/*
- * ***** BEGIN GPL LICENSE BLOCK *****
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list