pull_request doxygen#11332 Renamed option to UML_MAX_EDGE_LABELS

Some minor code simplications

Author: doxygen

src/config.xml

@@ -3924,15 +3924,13 @@ where `loc1` and `loc2` can be relative or absolute paths or URLs.
 ]]>
       </docs>
     </option>
-    <option type='int' id='UML_LIMIT_EDGE_FIELDS' defval='10' minval='0' maxval='100' depends='UML_LOOK'>
+    <option type='int' id='UML_MAX_EDGE_LABELS' defval='10' minval='0' maxval='100' depends='UML_LOOK'>
       <docs>
 <![CDATA[
- If the \ref cfg_uml_look "UML_LOOK" tag is enabled, fields are shown along
- the edge between 2 clas nodes.
- If there are many fields and many nodes the
- graph may become too big to be useful. The \c UML_LIMIT_EDGE_FIELDS
- threshold limits the number of items to make the size more
- manageable. Set this to 0 for no limit.
+ If the \ref cfg_uml_look "UML_LOOK" tag is enabled, field labels are shown along
+ the edge between two class nodes. If there are many fields and many nodes the
+ graph may become too cluttered. The \c UML_MAX_EDGE_LABELS threshold limits
+ the number of items to make the size more manageable. Set this to 0 for no limit.
 ]]>
       </docs>
     </option>

src/dotclassgraph.cpp

@@ -231,17 +231,20 @@ static QCString joinLabels(const StringSet &ss)
 {
   QCString label;
   int count=1;
-  int maxLabels = Config_getInt(UML_LIMIT_EDGE_FIELDS);
+  int maxLabels = Config_getInt(UML_MAX_EDGE_LABELS);
   auto it = std::begin(ss), e = std::end(ss);
   if (it!=e) // set not empty
   {
-    label += (*it++).c_str();
-    for (; it!=e && (maxLabels == 0 || count < maxLabels) ; ++it,++count)
+    label = *it++;
+    for (; it!=e && (maxLabels==0 || count<maxLabels) ; ++it,++count)
     {
       label += '\n';
-      label += (*it).c_str();
+      label += *it;
+    }
+    if (maxLabels!=0 && count==maxLabels)
+    {
+      label+="\n...";
     }
-    if (maxLabels != 0 && count==maxLabels) label+="\n...";
   }
   return label;
 }