summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKristaps Dzonsons <kristaps@bsd.lv>2009-03-08 18:50:40 +0000
committerKristaps Dzonsons <kristaps@bsd.lv>2009-03-08 18:50:40 +0000
commit80c830e4229bab0c1bc203bac882a038109a098f (patch)
treef22d5135f0bfff659b67ed73128c47a9ec61a4fe
parent615c4124c25bfd89a89bc7b14e0fe58520858cb1 (diff)
downloadmandoc-80c830e4229bab0c1bc203bac882a038109a098f.tar.gz
Removed static limitation of buffer size.
-rw-r--r--mmain.c46
1 files changed, 20 insertions, 26 deletions
diff --git a/mmain.c b/mmain.c
index 8f57a8c1..c823901f 100644
--- a/mmain.c
+++ b/mmain.c
@@ -30,7 +30,7 @@
#include "mmain.h"
-#define MD_LINE_SZ (256) /* Max input line size. */
+#define MD_LINE_SZ (256) /* Input line step-size. */
struct mmain {
int warn; /* Warning flags. */
@@ -269,39 +269,31 @@ optswarn(struct mmain *p, char *arg)
static int
parse(struct mmain *p)
{
- ssize_t sz, i;
- size_t pos;
- char line[MD_LINE_SZ];
- int lnn;
-
- /*
- * This is a little more complicated than fgets. TODO: have
- * some benchmarks that show it's faster (note that I want to
- * check many, many manuals simultaneously, so speed is
- * important). Fill a buffer (sized to the block size) with a
- * single read, then parse \n-terminated lines into a line
- * buffer, which is passed to the parser. Hard-code the line
- * buffer to a particular size -- a reasonable assumption.
- */
-
- for (lnn = 1, pos = 0; ; ) {
+ ssize_t sz;
+ int i, pos, len, lnn;
+ char *line;
+
+ for (line = NULL, lnn = 1, len = pos = 0; ; ) {
if (-1 == (sz = read(p->fdin, p->buf, p->bufsz))) {
warn("%s", p->in);
return(0);
} else if (0 == sz)
break;
- for (i = 0; i < sz; i++) {
+ for (i = 0; i < (int)sz; i++) {
+ if (pos >= len) {
+ len += MD_LINE_SZ;
+ line = realloc(line, len);
+ if (NULL == line)
+ err(1, "realloc");
+ }
+
if ('\n' != p->buf[i]) {
- if (pos < sizeof(line)) {
- line[(int)pos++] = p->buf[(int)i];
- continue;
- }
- warnx("%s: line %d too long", p->in, lnn);
- return(0);
+ line[pos++] = p->buf[i];
+ continue;
}
-
- line[(int)pos] = 0;
+
+ line[pos] = 0;
if ( ! mdoc_parseln(p->mdoc, lnn, line))
return(0);
@@ -310,6 +302,8 @@ parse(struct mmain *p)
}
}
+ if (line)
+ free(line);
return(mdoc_endparse(p->mdoc));
}