Slight optimisation: Don't check for duplication if we won't use the result anyway.

1 files changed, 7 insertions, 7 deletions

diff --git a/OpenSim/Region/Physics/OdePlugin/OdePlugin.cs b/OpenSim/Region/Physics/OdePlugin/OdePlugin.cs
index a250a6a..f735f60 100644
--- a/OpenSim/Region/Physics/OdePlugin/OdePlugin.cs
+++ b/OpenSim/Region/Physics/OdePlugin/OdePlugin.cs
@@ -817,18 +817,18 @@ namespace OpenSim.Region.Physics.OdePlugin
                 // appears to be phantom for the world
                 Boolean skipThisContact = false;
 
-                if (contacts[i].depth < 0f)
-                    skipThisContact = true;
-
-                if (checkDupe(contacts[i], p2.PhysicsActorType))
-                    skipThisContact = true;
-
                 if ((p1 is OdePrim) && (((OdePrim)p1).m_isVolumeDetect))
                     skipThisContact = true;   // No collision on volume detect prims
 
-                if ((p2 is OdePrim) && (((OdePrim)p2).m_isVolumeDetect))
+                if (!skipThisContact && (p2 is OdePrim) && (((OdePrim)p2).m_isVolumeDetect))
                     skipThisContact = true;   // No collision on volume detect prims
 
+                if (!skipThisContact && contacts[i].depth < 0f)
+                    skipThisContact = true;
+
+                if (!skipThisContact && checkDupe(contacts[i], p2.PhysicsActorType))
+                    skipThisContact = true;
+
                 if (!skipThisContact)
                 {
                     // If we're colliding against terrain