Fix Issue 13212 - Windows line endings handled incorrectly by ddoc ma…

…cros Handle \r as \n in macro definitions to prevent a trailing carriage return on a macro with an empty line following it.
dlang · Jul 29, 2014 · 35735aa · 35735aa
1 parent ecb09de
commit 35735aa
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/src/doc.c b/src/doc.c
@@ -1779,6 +1779,7 @@ void DocComment::parseMacros(Escape **pescapetable, Macro **pmacrotable, const u
                     p++;
                     continue;
 
+                case '\r':
                 case '\n':
                     p++;
                     goto Lcont;
@@ -1843,14 +1844,10 @@ void DocComment::parseMacros(Escape **pescapetable, Macro **pmacrotable, const u
         textstart = p;
 
       Ltext:
-        while (p < pend && *p != '\n')
+        while (p < pend && *p != '\r' && *p != '\n')
             p++;
         textlen = p - textstart;
 
-        // Remove trailing \r if there is one
-        if (p > m && p[-1] == '\r')
-            textlen--;
-
         p++;
         //printf("p = %p, pend = %p\n", p, pend);
 
@@ -1859,8 +1856,8 @@ void DocComment::parseMacros(Escape **pescapetable, Macro **pmacrotable, const u
 
      Lskipline:
         // Ignore this line
-        while (p < pend && *p++ != '\n')
-            ;
+        while (p < pend && *p != '\r' && *p != '\n')
+            p++;
     }
 Ldone:
     if (namelen)

diff --git a/test/compilable/extra-files/ddoc3.ddoc b/test/compilable/extra-files/ddoc3.ddoc
@@ -1 +1,3 @@
-HELLO = world
+HELLO = world
+
+UNUSED=unused