Actually this empty() check here is redundant.

ResortChildren() does that at the very beginning.

Change-Id: I696c9d796cc6631048fb3d9409d6eda1085a7612

svtools/source/contnr/treelist.cxx

@@ -1541,8 +1541,7 @@ void SvTreeList::ResortChildren( SvTreeListEntry* pParent )
     for (; it != itEnd; ++it)
     {
         SvTreeListEntry& r = *it;
-        if (!r.maChildren.empty())
-            ResortChildren(&r);
+        ResortChildren(&r);
     }
 
     SetListPositions(pParent->maChildren); // correct list position in target list