diff -r 17408c3607ce -r f25ba6fd7a08 src/ccodegen.c --- 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, ""); } @@ -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, "/"); 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, ""); } } @@ -90,21 +89,21 @@ "<"); } else if (c == '\"') { if (parseinclude) { - ucx_bprintf(dest, "\">%.*s\"", - includefile.length, includefile.ptr); + ucx_buffer_puts(dest, "\">"); + ucx_buffer_write(ifilebuf->space, 1, ifilebuf->size, dest); + ucx_buffer_puts(dest, "\""); parseinclude = 0; } else { ucx_buffer_puts(dest, "') { ucx_buffer_puts(dest, ">"); } 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, ""); } } - 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); } }