+ implement rich compare protocol of Python interface for Placement & Rotation

src/Base/PlacementPy.xml

@@ -9,7 +9,8 @@
 		FatherInclude="Base/PyObjectBase.h" 
 		Namespace="Base" 
 		Constructor="true"
-		Delete="true" 
+        Delete="true"
+        RichCompare="true"
 		FatherNamespace="Base">
 	<Documentation>
 		<Author Licence="LGPL" Name="Juergen Riegel" EMail="FreeCAD@juergen-riegel.net" />
@@ -75,7 +76,7 @@ Placement(Base, Axis, Angle) -- define position and rotation
 				</UserDocu>
 			</Documentation>
 		</Methode>
-                <Methode Name="isNull" Const="true">
+        <Methode Name="isNull" Const="true">
 			<Documentation>
 				<UserDocu>
 					isNull() -> Bool
@@ -83,7 +84,7 @@ Placement(Base, Axis, Angle) -- define position and rotation
 				</UserDocu>
 			</Documentation>
 		</Methode>
-		<Attribute Name="Base" ReadOnly="false">
+        <Attribute Name="Base" ReadOnly="false">
 			<Documentation>
 				<UserDocu>Vector to the Base Position of the Placement</UserDocu>
 			</Documentation>

src/Base/PlacementPyImp.cpp

@@ -118,6 +118,37 @@ int PlacementPy::PyInit(PyObject* args, PyObject* /*kwd*/)
     return -1;
 }
 
+PyObject* PlacementPy::richCompare(PyObject *v, PyObject *w, int op)
+{
+    if (PyObject_TypeCheck(v, &(PlacementPy::Type)) &&
+        PyObject_TypeCheck(w, &(PlacementPy::Type))) {
+        Base::Placement p1 = *static_cast<PlacementPy*>(v)->getPlacementPtr();
+        Base::Placement p2 = *static_cast<PlacementPy*>(w)->getPlacementPtr();
+
+        PyObject *res=0;
+        if (op != Py_EQ && op != Py_NE) {
+            PyErr_SetString(PyExc_TypeError,
+            "no ordering relation is defined for Placement");
+            return 0;
+        }
+        else if (op == Py_EQ) {
+            res = (p1 == p2) ? Py_True : Py_False;
+            Py_INCREF(res);
+            return res;
+        }
+        else {
+            res = (p1 != p2) ? Py_True : Py_False;
+            Py_INCREF(res);
+            return res;
+        }
+    }
+    else {
+        // This always returns False
+        Py_INCREF(Py_NotImplemented);
+        return Py_NotImplemented;
+    }
+}
+
 PyObject* PlacementPy::move(PyObject * args)
 {
     PyObject *vec;

src/Base/RotationPy.xml

@@ -10,6 +10,7 @@
 		  Namespace="Base"
 		  Constructor="true"
 		  Delete="true"
+          RichCompare="true"
 		  FatherNamespace="Base">
 		<Documentation>
 			<Author Licence="LGPL" Name="Juergen Riegel" EMail="FreeCAD@juergen-riegel.net" />
@@ -59,7 +60,7 @@
 				</UserDocu>
 			</Documentation>
 		</Methode>
-                <Methode Name="isNull" Const="true">
+        <Methode Name="isNull" Const="true">
 			<Documentation>
 				<UserDocu>
 					isNull() -> Bool
@@ -67,7 +68,7 @@
 				</UserDocu>
 			</Documentation>
 		</Methode>
-		<Attribute Name="Q" ReadOnly="false">
+        <Attribute Name="Q" ReadOnly="false">
 			<Documentation>
 				<UserDocu>The rotation elements (as quaternion)</UserDocu>
 			</Documentation>

src/Base/RotationPyImp.cpp

@@ -77,14 +77,14 @@ int RotationPy::PyInit(PyObject* args, PyObject* /*kwd*/)
     PyErr_Clear();
     double angle;
     if (PyArg_ParseTuple(args, "O!d", &(Base::VectorPy::Type), &o, &angle)) {
-      // NOTE: The last parameter defines the rotation angle in degree.
+      // NOTE: The last parameter defines the rotation angle in degree.
       getRotationPtr()->setValue(static_cast<Base::VectorPy*>(o)->value(), Base::toRadians<double>(angle));
       return 0;
     }
 
     PyErr_Clear();
     if (PyArg_ParseTuple(args, "O!d", &(Base::MatrixPy::Type), &o, &angle)) {
-      // NOTE: The last parameter defines the rotation angle in degree.
+      // NOTE: The last parameter defines the rotation angle in degree.
       getRotationPtr()->setValue(static_cast<Base::MatrixPy*>(o)->value());
       return 0;
     }
@@ -164,6 +164,37 @@ int RotationPy::PyInit(PyObject* args, PyObject* /*kwd*/)
     return -1;
 }
 
+PyObject* RotationPy::richCompare(PyObject *v, PyObject *w, int op)
+{
+    if (PyObject_TypeCheck(v, &(RotationPy::Type)) &&
+        PyObject_TypeCheck(w, &(RotationPy::Type))) {
+        Base::Rotation r1 = *static_cast<RotationPy*>(v)->getRotationPtr();
+        Base::Rotation r2 = *static_cast<RotationPy*>(w)->getRotationPtr();
+
+        PyObject *res=0;
+        if (op != Py_EQ && op != Py_NE) {
+            PyErr_SetString(PyExc_TypeError,
+            "no ordering relation is defined for Rotation");
+            return 0;
+        }
+        else if (op == Py_EQ) {
+            res = (r1 == r2) ? Py_True : Py_False;
+            Py_INCREF(res);
+            return res;
+        }
+        else {
+            res = (r1 != r2) ? Py_True : Py_False;
+            Py_INCREF(res);
+            return res;
+        }
+    }
+    else {
+        // This always returns False
+        Py_INCREF(Py_NotImplemented);
+        return Py_NotImplemented;
+    }
+}
+
 PyObject* RotationPy::invert(PyObject * args)
 {
     if (!PyArg_ParseTuple(args, ""))
@@ -205,16 +236,16 @@ PyObject* RotationPy::toEuler(PyObject * args)
     return Py::new_reference_to(tuple);
 }
 
-PyObject* RotationPy::isNull(PyObject *args)
-{
-    if (!PyArg_ParseTuple(args, ""))
-        return NULL;
-    Base::Rotation rot = * getRotationPtr();
-    Base::Rotation nullrot(0,0,0,1);
-    Base::Rotation nullrotinv(0,0,0,-1);
-    bool null = (rot == nullrot) | (rot == nullrotinv);
-    return Py_BuildValue("O", (null ? Py_True : Py_False));
-}
+PyObject* RotationPy::isNull(PyObject *args)
+{
+    if (!PyArg_ParseTuple(args, ""))
+        return NULL;
+    Base::Rotation rot = * getRotationPtr();
+    Base::Rotation nullrot(0,0,0,1);
+    Base::Rotation nullrotinv(0,0,0,-1);
+    bool null = (rot == nullrot) | (rot == nullrotinv);
+    return Py_BuildValue("O", (null ? Py_True : Py_False));
+}
 
 Py::Tuple RotationPy::getQ(void) const
 {