From da09a1bfd86c2066ab492178bd2c0faa96bfd579 Mon Sep 17 00:00:00 2001 From: Michael Mann Date: Fri, 26 Jun 2015 23:00:33 -0400 Subject: Take all epan_column_info members that always get allocated to "number of columns" to share their own data structure. Change-Id: Ib982662db6cf68730a7d121eac60d9bc5ae67429 Reviewed-on: https://code.wireshark.org/review/9195 Petri-Dish: Michael Mann Reviewed-by: Anders Broman Reviewed-by: Michael Mann --- tfshark.c | 38 ++++++++++++++++++++------------------ 1 file changed, 20 insertions(+), 18 deletions(-) (limited to 'tfshark.c') diff --git a/tfshark.c b/tfshark.c index f3d892a31b..05f1603e2a 100644 --- a/tfshark.c +++ b/tfshark.c @@ -2254,21 +2254,23 @@ print_columns(capture_file *cf) size_t buf_offset; size_t column_len; size_t col_len; + col_item_t* col_item; line_bufp = get_line_buf(256); buf_offset = 0; *line_bufp = '\0'; for (i = 0; i < cf->cinfo.num_cols; i++) { + col_item = &cf->cinfo.columns[i]; /* Skip columns not marked as visible. */ if (!get_column_visible(i)) continue; - switch (cf->cinfo.col_fmt[i]) { + switch (col_item->col_fmt) { case COL_NUMBER: - column_len = col_len = strlen(cf->cinfo.col_data[i]); + column_len = col_len = strlen(col_item->col_data); if (column_len < 3) column_len = 3; line_bufp = get_line_buf(buf_offset + column_len); - put_spaces_string(line_bufp + buf_offset, cf->cinfo.col_data[i], col_len, column_len); + put_spaces_string(line_bufp + buf_offset, col_item->col_data, col_len, column_len); break; case COL_CLS_TIME: @@ -2279,11 +2281,11 @@ print_columns(capture_file *cf) case COL_UTC_TIME: case COL_UTC_YMD_TIME: /* XXX - wider */ case COL_UTC_YDOY_TIME: /* XXX - wider */ - column_len = col_len = strlen(cf->cinfo.col_data[i]); + column_len = col_len = strlen(col_item->col_data); if (column_len < 10) column_len = 10; line_bufp = get_line_buf(buf_offset + column_len); - put_spaces_string(line_bufp + buf_offset, cf->cinfo.col_data[i], col_len, column_len); + put_spaces_string(line_bufp + buf_offset, col_item->col_data, col_len, column_len); break; case COL_DEF_SRC: @@ -2295,11 +2297,11 @@ print_columns(capture_file *cf) case COL_DEF_NET_SRC: case COL_RES_NET_SRC: case COL_UNRES_NET_SRC: - column_len = col_len = strlen(cf->cinfo.col_data[i]); + column_len = col_len = strlen(col_item->col_data); if (column_len < 12) column_len = 12; line_bufp = get_line_buf(buf_offset + column_len); - put_spaces_string(line_bufp + buf_offset, cf->cinfo.col_data[i], col_len, column_len); + put_spaces_string(line_bufp + buf_offset, col_item->col_data, col_len, column_len); break; case COL_DEF_DST: @@ -2311,17 +2313,17 @@ print_columns(capture_file *cf) case COL_DEF_NET_DST: case COL_RES_NET_DST: case COL_UNRES_NET_DST: - column_len = col_len = strlen(cf->cinfo.col_data[i]); + column_len = col_len = strlen(col_item->col_data); if (column_len < 12) column_len = 12; line_bufp = get_line_buf(buf_offset + column_len); - put_string_spaces(line_bufp + buf_offset, cf->cinfo.col_data[i], col_len, column_len); + put_string_spaces(line_bufp + buf_offset, col_item->col_data, col_len, column_len); break; default: - column_len = strlen(cf->cinfo.col_data[i]); + column_len = strlen(col_item->col_data); line_bufp = get_line_buf(buf_offset + column_len); - put_string(line_bufp + buf_offset, cf->cinfo.col_data[i], column_len); + put_string(line_bufp + buf_offset, col_item->col_data, column_len); break; } buf_offset += column_len; @@ -2341,12 +2343,12 @@ print_columns(capture_file *cf) * even if we're only adding " ". */ line_bufp = get_line_buf(buf_offset + 4); - switch (cf->cinfo.col_fmt[i]) { + switch (col_item->col_fmt) { case COL_DEF_SRC: case COL_RES_SRC: case COL_UNRES_SRC: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_DST: case COL_RES_DST: @@ -2365,7 +2367,7 @@ print_columns(capture_file *cf) case COL_DEF_DL_SRC: case COL_RES_DL_SRC: case COL_UNRES_DL_SRC: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_DL_DST: case COL_RES_DL_DST: @@ -2384,7 +2386,7 @@ print_columns(capture_file *cf) case COL_DEF_NET_SRC: case COL_RES_NET_SRC: case COL_UNRES_NET_SRC: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_NET_DST: case COL_RES_NET_DST: @@ -2403,7 +2405,7 @@ print_columns(capture_file *cf) case COL_DEF_DST: case COL_RES_DST: case COL_UNRES_DST: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_SRC: case COL_RES_SRC: @@ -2422,7 +2424,7 @@ print_columns(capture_file *cf) case COL_DEF_DL_DST: case COL_RES_DL_DST: case COL_UNRES_DL_DST: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_DL_SRC: case COL_RES_DL_SRC: @@ -2441,7 +2443,7 @@ print_columns(capture_file *cf) case COL_DEF_NET_DST: case COL_RES_NET_DST: case COL_UNRES_NET_DST: - switch (cf->cinfo.col_fmt[i + 1]) { + switch (cf->cinfo.columns[i+1].col_fmt) { case COL_DEF_NET_SRC: case COL_RES_NET_SRC: -- cgit v1.2.1