This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix ptype problem printing typedefs defined differently in different compilation units


On Tuesday 03 January 2006 22:45, Jim Blandy wrote:

> what would break if parse_exp_in_context used the current source
> position as a fallback when get_selected_block returns zero?  It
> seems reasonable to me.

That seems like the most reasonable alternative to me as well.  The
following patch solves the same problem as the original patch, in what
is probably a little cleaner way.  I don't get any testsuite
regressions when using it.

-Fred

2006-01-04  Fred Fish  <fnf@fishfood.ninemoons.com>

	* parse.c (source.h): Include.
	(parse_exp_in_context):  Use static source context if no
	other context found.

Index: parse.c
===================================================================
RCS file: /cvsroots/latest/src/gdb/gdb/parse.c,v
retrieving revision 1.1.1.2
diff -c -p -r1.1.1.2 parse.c
*** parse.c	30 Dec 2005 18:53:04 -0000	1.1.1.2
--- parse.c	4 Jan 2006 18:35:57 -0000
***************
*** 52,57 ****
--- 52,58 ----
  #include "doublest.h"
  #include "gdb_assert.h"
  #include "block.h"
+ #include "source.h"
  
  /* Standard set of definitions for printing, dumping, prefixifying,
   * and evaluating expressions.  */
*************** parse_exp_in_context (char **stringptr, 
*** 1075,1087 ****
    old_chain = make_cleanup (free_funcalls, 0 /*ignore*/);
    funcall_chain = 0;
  
    if (block)
      {
        expression_context_block = block;
        expression_context_pc = BLOCK_START (block);
      }
-   else
-     expression_context_block = get_selected_block (&expression_context_pc);
  
    expout_size = 10;
    expout_ptr = 0;
--- 1076,1102 ----
    old_chain = make_cleanup (free_funcalls, 0 /*ignore*/);
    funcall_chain = 0;
  
+   /* If no context specified, try using the current frame, if any. */
+ 
+   if (!block)
+     block = get_selected_block (&expression_context_pc);
+ 
+   /* Fall back to using the current source static context, if any. */
+ 
+   if (!block)
+     {
+       struct symtab_and_line cursal = get_current_source_symtab_and_line ();
+       if (cursal.symtab)
+ 	block = BLOCKVECTOR_BLOCK (BLOCKVECTOR (cursal.symtab), STATIC_BLOCK);
+     }
+ 
+   /* Save the context, if specified by caller, or found above. */
+ 
    if (block)
      {
        expression_context_block = block;
        expression_context_pc = BLOCK_START (block);
      }
  
    expout_size = 10;
    expout_ptr = 0;




Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]