(related: tdf#80715) sw: fix Ring::MoveTo() not doing anything...

... if the parameter is currently in the same list.

The "boost::intrusive::circular_list_algorithms::transfer" has a
precondition that the 2 parameters must not be in the same list.

This causes an infinite loop in SwFindParaText::Find(), which is hiding
the infinite loop that i'm trying to debug...

While at it, remove some unnecessary complexity.

Change-Id: Ib41f52c6d5c44ecc358c6170ee1e6e98729e1302

1 files changed, 3 insertions, 7 deletions

diff --git a/sw/inc/ring.hxx b/sw/inc/ring.hxx
index e0bfc5a30b90..9f70061e9725 100644
--- a/sw/inc/ring.hxx
+++ b/sw/inc/ring.hxx
@@ -135,16 +135,12 @@ namespace sw
     inline void Ring<value_type>::MoveTo(value_type* pDestRing)
     {
         value_type* pThis = static_cast< value_type* >(this);
+        algo::unlink(pThis);
         // insert into "new"
-        if( pDestRing )
+        if (pDestRing)
         {
-            if(algo::unique(pThis))
-                algo::link_before(pDestRing, pThis);
-            else
-                algo::transfer(pDestRing, pThis);
+            algo::link_before(pDestRing, pThis);
         }
-        else
-            algo::unlink(pThis);
     }
 
     /**