From a6c47c143af59ad4f499d25ed6713a3b9b4bb67a Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Sat, 15 Dec 2001 05:18:42 +0000 Subject: [PATCH] * ui-out.c (struct ui_out_table): Add field entry_level. (verify_field): New function. (verify_field_proper_position): Delete function. (verify_field_alignment): Delete function. (ui_out_field_int): Update to use verify_field. (ui_out_field_skip): Ditto. (ui_out_field_string): Ditto. (ui_out_field_fmt): Ditto. (ui_out_table_begin): Initialize table.entry_level. (ui_out_table_end): Clear table.entry_level. (ui_out_begin): Call verify_field before pushing the new tuple or list onto the stack. Use table.entry_level. --- gdb/ChangeLog | 15 ++++++++ gdb/ui-out.c | 104 ++++++++++++++++++++++++++------------------------ 2 files changed, 69 insertions(+), 50 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ed9fb46a79..a4d5b179ee 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,18 @@ +2001-12-15 Andrew Cagney + + * ui-out.c (struct ui_out_table): Add field entry_level. + (verify_field): New function. + (verify_field_proper_position): Delete function. + (verify_field_alignment): Delete function. + (ui_out_field_int): Update to use verify_field. + (ui_out_field_skip): Ditto. + (ui_out_field_string): Ditto. + (ui_out_field_fmt): Ditto. + (ui_out_table_begin): Initialize table.entry_level. + (ui_out_table_end): Clear table.entry_level. + (ui_out_begin): Call verify_field before pushing the new tuple or + list onto the stack. Use table.entry_level. + 2001-12-14 Corinna Vinschen * config/djgpp/fnchange.lst: Add entries for opcodes/xstormy16-* files. diff --git a/gdb/ui-out.c b/gdb/ui-out.c index 0d49aba738..06db646de3 100644 --- a/gdb/ui-out.c +++ b/gdb/ui-out.c @@ -71,6 +71,11 @@ struct ui_out_table header is being generated. */ int body_flag; + /* The level at which each entry of the table is to be found. A row + (a tuple) is made up of entries. Consequently ENTRY_LEVEL is one + above that of the table. */ + int entry_level; + /* Number of table columns (as specified in the table_begin call). */ int columns; @@ -262,8 +267,8 @@ static void append_header_to_list (struct ui_out *uiout, int width, static int get_next_header (struct ui_out *uiout, int *colno, int *width, int *alignment, char **colhdr); static void clear_header_list (struct ui_out *uiout); -static void verify_field_proper_position (struct ui_out *uiout); -static void verify_field_alignment (struct ui_out *uiout, int fldno, int *width, int *alignment); +static void verify_field (struct ui_out *uiout, int *fldno, int *width, + int *align); static void init_ui_out_state (struct ui_out *uiout); @@ -283,6 +288,7 @@ previous table_end."); uiout->table.flag = 1; uiout->table.body_flag = 0; + uiout->table.entry_level = uiout->level + 1; uiout->table.columns = nbrofcols; if (tblid != NULL) uiout->table.id = xstrdup (tblid); @@ -322,6 +328,7 @@ ui_out_table_end (struct ui_out *uiout) internal_error (__FILE__, __LINE__, "misplaced table_end or missing table_begin."); + uiout->table.entry_level = 0; uiout->table.body_flag = 0; uiout->table.flag = 0; @@ -357,9 +364,29 @@ ui_out_begin (struct ui_out *uiout, internal_error (__FILE__, __LINE__, "table header or table_body expected; lists must be \ specified after table_body."); + + /* Be careful to verify the ``field'' before the new tuple/list is + pushed onto the stack. That way the containing list/table/row is + verified and not the newly created tuple/list. This verification + is needed (at least) for the case where a table row entry + contains either a tuple/list. For that case bookkeeping such as + updating the column count or advancing to the next heading still + needs to be performed. */ + { + int fldno; + int width; + int align; + verify_field (uiout, &fldno, &width, &align); + } + new_level = push_level (uiout, type, id); - if (uiout->table.flag && (new_level == 1)) + + /* If the push puts us at the same level as a table row entry, we've + got a new table row. Put the header pointer back to the start. */ + if (uiout->table.body_flag + && uiout->table.entry_level == new_level) uiout->table.header_next = uiout->table.header_first; + uo_begin (uiout, type, new_level, id); } @@ -456,12 +483,7 @@ ui_out_field_int (struct ui_out *uiout, int align; struct ui_out_level *current = current_level (uiout); - verify_field_proper_position (uiout); - - current->field_count += 1; - fldno = current->field_count; - - verify_field_alignment (uiout, fldno, &width, &align); + verify_field (uiout, &fldno, &width, &align); uo_field_int (uiout, fldno, width, align, fldname, value); } @@ -505,14 +527,8 @@ ui_out_field_skip (struct ui_out *uiout, int fldno; int width; int align; - struct ui_out_level *current = current_level (uiout); - - verify_field_proper_position (uiout); - - current->field_count += 1; - fldno = current->field_count; - verify_field_alignment (uiout, fldno, &width, &align); + verify_field (uiout, &fldno, &width, &align); uo_field_skip (uiout, fldno, width, align, fldname); } @@ -525,14 +541,8 @@ ui_out_field_string (struct ui_out *uiout, int fldno; int width; int align; - struct ui_out_level *current = current_level (uiout); - - verify_field_proper_position (uiout); - - current->field_count += 1; - fldno = current->field_count; - verify_field_alignment (uiout, fldno, &width, &align); + verify_field (uiout, &fldno, &width, &align); uo_field_string (uiout, fldno, width, align, fldname, string); } @@ -547,15 +557,9 @@ ui_out_field_fmt (struct ui_out *uiout, int fldno; int width; int align; - struct ui_out_level *current = current_level (uiout); - - verify_field_proper_position (uiout); - - current->field_count += 1; - fldno = current->field_count; /* will not align, but has to call anyway */ - verify_field_alignment (uiout, fldno, &width, &align); + verify_field (uiout, &fldno, &width, &align); va_start (args, format); @@ -1055,39 +1059,37 @@ get_next_header (struct ui_out *uiout, return 1; } -/* makes sure the field_* calls were properly placed */ + +/* Verify that the field/tuple/list is correctly positioned. Return + the field number and corresponding alignment (if + available/applicable). */ static void -verify_field_proper_position (struct ui_out *uiout) +verify_field (struct ui_out *uiout, int *fldno, int *width, int *align) { + struct ui_out_level *current = current_level (uiout); + char *text; + if (uiout->table.flag) { if (!uiout->table.body_flag) internal_error (__FILE__, __LINE__, "table_body missing; table fields must be \ specified after table_body and inside a list."); - if (uiout->level == 0) - internal_error (__FILE__, __LINE__, - "list_begin missing; table fields must be \ -specified after table_body and inside a list."); + /* NOTE: cagney/2001-12-08: There was a check here to ensure + that this code was only executed when uiout->level was + greater than zero. That no longer applies - this code is run + before each table row tuple is started and at that point the + level is zero. */ } -} -/* determines what is the alignment policy */ - -static void -verify_field_alignment (struct ui_out *uiout, - int fldno, - int *width, - int *align) -{ - int colno; - char *text; + current->field_count += 1; - if (uiout->table.flag - && get_next_header (uiout, &colno, width, align, &text)) + if (uiout->table.body_flag + && uiout->table.entry_level == uiout->level + && get_next_header (uiout, fldno, width, align, &text)) { - if (fldno != colno) + if (*fldno != current->field_count) internal_error (__FILE__, __LINE__, "ui-out internal error in handling headers."); } @@ -1095,9 +1097,11 @@ verify_field_alignment (struct ui_out *uiout, { *width = 0; *align = ui_noalign; + *fldno = current->field_count; } } + /* access to ui_out format private members */ void -- 2.34.1