fix(markdown): fix -Wcomma warning, improve formatting

warning: possible misuse of comma operator here [-Wcomma]

Author: artyom-fedosov

src/markdown.cpp

@@ -768,7 +768,11 @@ size_t Markdown::Private::findEmphasisChar(std::string_view data, char c, size_t
     if (data[i]=='`')
     {
       int snb=0;
-      while (i<size && data[i]=='`') snb++,i++;
+      while (i < size && data[i] == '`')
+      {
+        snb++;
+        i++;
+      }
 
       // find same pattern to end the span
       int enb=0;
@@ -967,11 +971,13 @@ int Markdown::Private::processNmdash(std::string_view data,size_t offset)
   int count=1;
   if (i<size && data[i]=='-') // found --
   {
-    count++,i++;
+    count++;
+    i++;
   }
   if (i<size && data[i]=='-') // found ---
   {
-    count++,i++;
+    count++;
+    i++;
   }
   if (i<size && data[i]=='-') // found ----
   {
@@ -1035,7 +1041,11 @@ int Markdown::Private::processHtmlTagWrite(std::string_view data,size_t offset,b
   size_t i=1;
   size_t l=0;
   // compute length of the tag name
-  while (i<size && isIdChar(data[i])) i++,l++;
+  while (i < size && isIdChar(data[i]))
+  {
+    i++;
+    l++;
+  }
   QCString tagName(data.substr(1,i-1));
   if (tagName.lower()=="pre") // found <pre> tag
   {
@@ -1885,7 +1895,11 @@ int Markdown::Private::isHeaderline(std::string_view data, bool allowAdjustLevel
   // test of level 1 header
   if (data[i]=='=')
   {
-    while (i<size && data[i]=='=') i++,c++;
+    while (i < size && data[i] == '=')
+    {
+      i++;
+      c++;
+    }
     while (i<size && data[i]==' ') i++;
     int level = (c>1 && (i>=size || data[i]=='\n')) ? 1 : 0;
     if (allowAdjustLevel && level==1 && indentLevel==-1)
@@ -1903,7 +1917,11 @@ int Markdown::Private::isHeaderline(std::string_view data, bool allowAdjustLevel
   // test of level 2 header
   if (data[i]=='-')
   {
-    while (i<size && data[i]=='-') i++,c++;
+    while (i < size && data[i] == '-')
+    {
+      i++;
+      c++;
+    }
     while (i<size && data[i]==' ') i++;
     return (c>1 && (i>=size || data[i]=='\n')) ? indentLevel+2 : 0;
   }
@@ -2107,12 +2125,20 @@ int Markdown::Private::isAtxHeader(std::string_view data,
   {
     return 0;
   }
-  while (i<size && data[i]=='#') i++,level++;
+  while (i < size && data[i] == '#')
+  {
+    i++;
+    level++;
+  }
   if (level>SectionType::MaxLevel) // too many #'s -> no section
   {
     return 0;
   }
-  while (i<size && data[i]==' ') i++,blanks++;
+  while (i < size && data[i] == ' ')
+  {
+    i++;
+    blanks++;
+  }
   if (level==1 && blanks==0)
   {
     return 0; // special case to prevent #someid seen as a header (see bug 671395)
@@ -2245,7 +2271,8 @@ static size_t computeIndentExcludingListMarkers(std::string_view data)
     { // end of indent
       break;
     }
-    indent++,i++;
+    indent++;
+    i++;
   }
   AUTO_TRACE_EXIT("result={}",indent);
   return indent;
@@ -2303,15 +2330,23 @@ static bool isFencedCodeBlock(std::string_view data,size_t refIndent,
   size_t indent=0;
   int startTildes=0;
   const size_t size = data.size();
-  while (i<size && data[i]==' ') indent++,i++;
+  while (i < size && data[i] == ' ')
+  {
+    indent++;
+    i++;
+  }
   if (indent>=refIndent+4)
   {
     AUTO_TRACE_EXIT("result=false: content is part of code block indent={} refIndent={}",indent,refIndent);
     return FALSE;
   } // part of code block
   char tildaChar='~';
   if (i<size && data[i]=='`') tildaChar='`';
-  while (i<size && data[i]==tildaChar) startTildes++,i++;
+  while (i < size && data[i] == tildaChar)
+  {
+    startTildes++;
+    i++;
+  }
   if (startTildes<3)
   {
     AUTO_TRACE_EXIT("result=false: no fence marker found #tildes={}",startTildes);
@@ -2357,7 +2392,11 @@ static bool isFencedCodeBlock(std::string_view data,size_t refIndent,
     {
       end=i;
       int endTildes=0;
-      while (i<size && data[i]==tildaChar) endTildes++,i++;
+      while (i < size && data[i] == tildaChar)
+      {
+        endTildes++;
+        i++;
+      }
       while (i<size && data[i]==' ') i++;
       {
         if (endTildes==startTildes)
@@ -2382,7 +2421,11 @@ static bool isCodeBlock(std::string_view data, size_t offset,size_t &indent)
   size_t i=0;
   size_t indent0=0;
   const size_t size = data.size();
-  while (i<size && data[i]==' ') indent0++,i++;
+  while (i < size && data[i] == ' ')
+  {
+    indent0++;
+    i++;
+  }
 
   if (indent0<codeBlockIndent)
   {
@@ -2474,7 +2517,11 @@ static size_t findTableColumns(std::string_view data,size_t &start,size_t &end,s
   size_t i=0,n=0;
   // find start character of the table line
   while (i<size && data[i]==' ') i++;
-  if (i<size && data[i]=='|' && data[i]!='\n') i++,n++; // leading | does not count
+  if (i < size && data[i] == '|' && data[i] != '\n')
+  {
+    i++;
+    n++; // leading | does not count
+  }
   start = i;
 
   // find end character of the table line
@@ -2484,7 +2531,11 @@ static size_t findTableColumns(std::string_view data,size_t &start,size_t &end,s
 
   if (j>0 && i>0) i--; // move i to point before newline
   while (i>0 && data[i]==' ') i--;
-  if (i>0 && data[i-1]!='\\' && data[i]=='|') i--,n++; // trailing or escaped | does not count
+  if (i > 0 && data[i - 1] != '\\' && data[i] == '|')
+  {
+    i--;
+    n++; // trailing or escaped | does not count
+  }
   end = i;
 
   // count columns between start and end
@@ -2970,7 +3021,11 @@ bool skipOverFileAndLineCommands(std::string_view data,size_t indent,size_t &off
       i+=9;
       location=data.substr(locStart,i-locStart);
       location+='\n';
-      while (indent>0 && i<size && data[i]==' ') i++,indent--;
+      while (indent > 0 && i < size && data[i] == ' ')
+      {
+        i++;
+        indent--;
+      }
       if (i<size && data[i]=='\n') i++;
       offset = i;
       return true;
@@ -2995,7 +3050,11 @@ size_t Markdown::Private::writeCodeBlock(std::string_view data,size_t refIndent)
     while (end<=size && data[end-1]!='\n') end++;
     size_t j=i;
     size_t indent=0;
-    while (j<end && data[j]==' ') j++,indent++;
+    while (j < end && data[j] == ' ')
+    {
+      j++;
+      indent++;
+    }
     //printf("j=%d end=%d indent=%d refIndent=%d tabSize=%d data={%s}\n",
     //    j,end,indent,refIndent,Config_getInt(TAB_SIZE),qPrint(QCString(data+i).left(end-i-1)));
     if (j==end-1) // empty line
@@ -3101,12 +3160,20 @@ size_t Markdown::Private::findEndOfLine(std::string_view data,size_t offset)
     }
     else if (nb==0 && data[end-1]=='`')
     {
-      while (end<=size && data[end-1]=='`') end++,nb++;
+      while (end <= size && data[end - 1] == '`')
+      {
+        end++;
+        nb++;
+      }
     }
     else if (nb>0 && data[end-1]=='`')
     {
       size_t enb=0;
-      while (end<=size && data[end-1]=='`') end++,enb++;
+      while (end <= size && data[end - 1] == '`')
+      {
+        end++;
+        enb++;
+      }
       if (enb==nb) nb=0;
     }
     else