Refactored code a bit

1 files changed, 10 insertions, 20 deletions

diff --git a/src/reflist.cpp b/src/reflist.cpp
index 59f7ad4..6f2a763 100644
--- a/src/reflist.cpp
+++ b/src/reflist.cpp
@@ -169,29 +169,19 @@ void RefList::generatePage()
     doc += item->prefix;
     doc += " \\_internalref ";
     doc += item->name;
-    doc += " \"";
     // escape \'s in title, see issue #5901
-    // prevent Obj-C names in e.g. todo list are seen as emoji
-    if (item->scope)
-    {
-      switch(item->scope->definitionType())
-      {
-        case Definition::TypeClass:
-        case Definition::TypeNamespace:
-        case Definition::TypeMember:
-        case Definition::TypePackage:
-          doc += substitute(substitute(item->title,"\\","\\\\"),":","∷");
-          break;
-        default:
-          doc += substitute(item->title,"\\","\\\\");
-          break;
-      }
-    }
-    else
+    QCString escapedTitle = substitute(item->title,"\\","\\\\");
+    if (item->scope &&
+        (item->scope->definitionType()==Definition::TypeClass ||
+         item->scope->definitionType()==Definition::TypeNamespace ||
+         item->scope->definitionType()==Definition::TypeMember ||
+         item->scope->definitionType()==Definition::TypePackage)
+       )
     {
-      doc += substitute(item->title,"\\","\\\\");
+      // prevent Obj-C names in e.g. todo list are seen as emoji
+      escapedTitle = substitute(escapedTitle,":","∷");
     }
-    doc += "\" ";
+    doc += " \""+escapedTitle+"\" ";
     // write declaration in case a function with arguments
     if (!item->args.isEmpty()) 
     {