diff options
author | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-01-03 13:59:21 +0000 |
---|---|---|
committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-01-03 13:59:21 +0000 |
commit | 49e1f5cf487fd2c297036e1873f357463817c13d (patch) | |
tree | 8bd125a7db62c2cc35df8279db30975d7bdd6963 /tbl.c | |
parent | e9da8c425b7c3bd1fb093fe12c25801c223ac95c (diff) | |
download | mandoc-49e1f5cf487fd2c297036e1873f357463817c13d.tar.gz mandoc-49e1f5cf487fd2c297036e1873f357463817c13d.tar.zst mandoc-49e1f5cf487fd2c297036e1873f357463817c13d.zip |
Make width calculations occur within tbl_term.c, not tbl.c. This allows
for front-ends to make decisions about widths, not the back-end.
To pull this off, first make each tbl_head contain a unique index value
(0 <= index < total tbl_head elements) and remove the tbl_calc() routine
from the back-end.
Then, when encountering the first tbl_span in the front-end, dynamically
create an array of configurations (termp_tbl) keyed on each tbl_head's
unique index value. Construct the decimals and widths at this time,
then continue parsing as before.
The termp_tbl and indexes are required because we pass a const tbl AST
into the front-end.
Diffstat (limited to 'tbl.c')
-rw-r--r-- | tbl.c | 162 |
1 files changed, 1 insertions, 161 deletions
@@ -1,4 +1,4 @@ -/* $Id: tbl.c,v 1.19 2011/01/02 20:34:05 kristaps Exp $ */ +/* $Id: tbl.c,v 1.20 2011/01/03 13:59:21 kristaps Exp $ */ /* * Copyright (c) 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv> * @@ -25,12 +25,6 @@ #include "libmandoc.h" #include "libroff.h" -static void tbl_calc(struct tbl_node *); -static void tbl_calc_data(struct tbl_node *, struct tbl_dat *); -static void tbl_calc_data_literal(struct tbl_dat *); -static void tbl_calc_data_number(struct tbl_node *, struct tbl_dat *); -static void tbl_calc_data_spanner(struct tbl_dat *); - enum rofferr tbl_read(struct tbl_node *tbl, int ln, const char *p, int offs) { @@ -151,162 +145,8 @@ tbl_end(struct tbl_node *tbl) if (NULL == tbl->first_span || NULL == tbl->first_span->first) TBL_MSG(tbl, MANDOCERR_TBLNODATA, tbl->line, tbl->pos); - else - tbl_calc(tbl); if (tbl->last_span) tbl->last_span->flags |= TBL_SPAN_LAST; } -static void -tbl_calc(struct tbl_node *tbl) -{ - struct tbl_span *sp; - struct tbl_dat *dp; - struct tbl_head *hp; - - /* Calculate width as the max of column cells' widths. */ - - for (sp = tbl->first_span; sp; sp = sp->next) { - switch (sp->pos) { - case (TBL_DATA_HORIZ): - /* FALLTHROUGH */ - case (TBL_DATA_DHORIZ): - continue; - default: - break; - } - for (dp = sp->first; dp; dp = dp->next) - tbl_calc_data(tbl, dp); - } - - /* Calculate width as the simple spanner value. */ - - for (hp = tbl->first_head; hp; hp = hp->next) - switch (hp->pos) { - case (TBL_HEAD_VERT): - hp->width = 1; - break; - case (TBL_HEAD_DVERT): - hp->width = 2; - break; - default: - break; - } -} - -static void -tbl_calc_data(struct tbl_node *tbl, struct tbl_dat *data) -{ - - if (NULL == data->layout) - return; - - /* Branch down into data sub-types. */ - - switch (data->layout->pos) { - case (TBL_CELL_HORIZ): - /* FALLTHROUGH */ - case (TBL_CELL_DHORIZ): - tbl_calc_data_spanner(data); - break; - case (TBL_CELL_LONG): - /* FALLTHROUGH */ - case (TBL_CELL_CENTRE): - /* FALLTHROUGH */ - case (TBL_CELL_LEFT): - /* FALLTHROUGH */ - case (TBL_CELL_RIGHT): - tbl_calc_data_literal(data); - break; - case (TBL_CELL_NUMBER): - tbl_calc_data_number(tbl, data); - break; - default: - abort(); - /* NOTREACHED */ - } -} - -static void -tbl_calc_data_spanner(struct tbl_dat *data) -{ - - /* N.B., these are horiz spanners (not vert) so always 1. */ - data->layout->head->width = 1; -} - -static void -tbl_calc_data_number(struct tbl_node *tbl, struct tbl_dat *data) -{ - int sz, d; - char *dp, pnt; - - /* - * First calculate number width and decimal place (last + 1 for - * no-decimal numbers). If the stored decimal is subsequent - * ours, make our size longer by that difference - * (right-"shifting"); similarly, if ours is subsequent the - * stored, then extend the stored size by the difference. - * Finally, re-assign the stored values. - */ - - /* TODO: use spacing modifier. */ - - assert(data->string); - sz = (int)strlen(data->string); - pnt = tbl->opts.decimal; - - if (NULL == (dp = strchr(data->string, pnt))) - d = sz + 1; - else - d = (int)(dp - data->string) + 1; - - sz += 2; - - if (data->layout->head->decimal > d) { - sz += data->layout->head->decimal - d; - d = data->layout->head->decimal; - } else - data->layout->head->width += - d - data->layout->head->decimal; - - if (sz > data->layout->head->width) - data->layout->head->width = sz; - if (d > data->layout->head->decimal) - data->layout->head->decimal = d; -} - -static void -tbl_calc_data_literal(struct tbl_dat *data) -{ - int sz, bufsz; - - /* - * Calculate our width and use the spacing, with a minimum - * spacing dictated by position (centre, e.g,. gets a space on - * either side, while right/left get a single adjacent space). - */ - - assert(data->string); - sz = (int)strlen(data->string); - - switch (data->layout->pos) { - case (TBL_CELL_LONG): - /* FALLTHROUGH */ - case (TBL_CELL_CENTRE): - bufsz = 2; - break; - default: - bufsz = 1; - break; - } - - if (data->layout->spacing) - bufsz = bufsz > data->layout->spacing ? - bufsz : data->layout->spacing; - - sz += bufsz; - if (data->layout->head->width < sz) - data->layout->head->width = sz; -} |