Mercurial > hg > c2html / file diff

--- a/src/ccodegen.c	Thu Aug 25 11:30:30 2016 +0200
+++ b/src/ccodegen.c	Thu Aug 25 12:16:57 2016 +0200
@@ -37,24 +37,23 @@
     "while", NULL
 };
 
-void c_highlighter(char *src, UcxBuffer *dest, int *multiline_comment) {
-    /* TODO: try to replace these buffers */
-    char wordbuf[WORDBUF_SIZE];
-    sstr_t word;
-    word.ptr = wordbuf; word.length = 0;
+void c_highlighter(char *src, UcxBuffer *dest, HighlighterData *hd) {
+    /* reset buffers without clearing them */
+    hd->primary_buffer->size = hd->primary_buffer->pos = 0;
+    hd->secondary_buffer->size = hd->secondary_buffer->pos = 0;
     
-    char includefilebuf[512];
-    sstr_t includefile;
-    includefile.ptr = includefilebuf;
-    includefile.length = 0;
+    /* alias the buffers for better handling */
+    UcxBuffer *wbuf = hd->primary_buffer;
+    UcxBuffer *ifilebuf = hd->secondary_buffer;
     
+    /* local information */
     size_t sp = (size_t)-1;
     int isstring = 0, iscomment = 0, isinclude = 0, parseinclude = 0;
     char quote = '\0';
     int isescaping = 0;
     
     /* continue a multi line comment highlighting */
-    if (*multiline_comment) {
+    if (hd->multiline_comment) {
         iscomment = 1;
         ucx_buffer_puts(dest, "<span class=\"c2html-comment\">");
     }
@@ -66,14 +65,14 @@
         
         /* comments */
         if (!isstring && c == '/') {
-            if (*multiline_comment && sp > 0 && src[sp-1] == '*') {
+            if (hd->multiline_comment && sp > 0 && src[sp-1] == '*') {
                 iscomment = 0;
-                *multiline_comment = 0;
+                hd->multiline_comment = 0;
                 ucx_buffer_puts(dest, "/</span>");
                 continue;
             } else if (!iscomment && (src[sp+1] == '/' || src[sp+1] == '*')) {
                 iscomment = 1;
-                *multiline_comment = (src[sp+1] == '*');
+                hd->multiline_comment = (src[sp+1] == '*');
                 ucx_buffer_puts(dest, "<span class=\"c2html-comment\">");
             }
         }
@@ -90,21 +89,21 @@
                         "<span class=\"c2html-stdinclude\">&lt;");
             } else if (c == '\"') {
                 if (parseinclude) {
-                    ucx_bprintf(dest, "\">%.*s\"</a>",
-                            includefile.length, includefile.ptr);
+                    ucx_buffer_puts(dest, "\">");
+                    ucx_buffer_write(ifilebuf->space, 1, ifilebuf->size, dest);
+                    ucx_buffer_puts(dest, "\"</a>");
                     parseinclude = 0;
                 } else {
                     ucx_buffer_puts(dest,
                             "<a class=\"c2html-userinclude\" href=\"");
-                    includefile.length = 0;
-                    includefile.ptr[includefile.length++] = '\"';
+                    ucx_buffer_putc(ifilebuf, '\"');
                     parseinclude = 1;
                 }
             } else if (c == '>') {
                 ucx_buffer_puts(dest,  "&gt;</span>");
             } else {
                 if (parseinclude) {
-                    includefile.ptr[includefile.length++] = c;
+                    ucx_buffer_putc(ifilebuf, c);
                 }
                 put_htmlescaped(dest, c);
             }
@@ -128,8 +127,10 @@
             } else {
                 if (isstring) {
                     put_htmlescaped(dest, c);
-                } else if (!isalnum(c) && c!='_' && c!='#' && c!='@') {
-                    if (word.length > 0 && word.length < WORDBUF_SIZE) {
+                } else if (!isalnum(c) && c!='_' && c!='#') {
+                    /* write buffered word, if any */
+                    if (wbuf->size > 0) {
+                        sstr_t word = sstrn(wbuf->space, wbuf->size);
                         int closespan = 1;
                         sstr_t typesuffix = ST("_t");
                         if (check_keyword(word, ckeywords)) {
@@ -153,20 +154,13 @@
                             ucx_buffer_puts(dest, "</span>");
                         }
                     }
-                    word.length = 0;
+                    wbuf->pos = wbuf->size = 0; /* reset word buffer */
+                    
+                    /* write current character */
                     put_htmlescaped(dest, c);
                 } else {
-                    /* read word */
-                    if (word.length < WORDBUF_SIZE) {
-                        word.ptr[word.length++] = c;
-                    } else if (word.length == WORDBUF_SIZE) {
-                        /* TODO: this will be removed */
-                        ucx_buffer_puts(dest,
-                                "!!! WARNING - WORD TOO LONG TO PARSE !!!");
-                        word.length = 0;
-                    } else {
-                        put_htmlescaped(dest, c);
-                    }
+                    /* buffer the current word */
+                    ucx_buffer_putc(wbuf, c);
                 }
             }