This is the mail archive of the gdb-patches@sources.redhat.com 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: RFA: switch to printf_unfiltered in symfile.c


Elena Zannoni <ezannoni@redhat.com> writes:

> Daniel Jacobowitz writes:
>  > On Wed, Oct 22, 2003 at 02:42:34PM -0400, J. Johnston wrote:
>  > > The following patch switches over the majority of the printfs in symfile.c 
>  > > to use printf_unfiltered.  This fixes a problem whereby if you attach to a 
>  > > process that has a lot of dynamic library references, you will get page 
>  > > breaks as it displays loading symbols messages.
>  > > 
>  > > I purposely left printf_filtered for info_ext_lang_command(), and 
>  > > list_overlays_command() as I felt that a user may want to examine the 
>  > > output and this should allow paging if necessary.
>  > > 
>  > > Ok to commit?
>  > > 
>  > > -- Jeff J.
>  > > 
>  > > 2003-10-22  Jeff Johnston  <jjohnstn@redhat.com>
>  > > 
>  > > 	* symfile.c (symbol_file_add_with_addrs_or_offsets): Switch to use
>  > > 	printf_unfiltered instead of printf_filtered for output messages.
>  > > 	(add_symbol_file_command): Ditto.
>  > > 	(reread_symbols): Ditto.
>  > > 	(overlay_auto_command): Ditto.
>  > > 	(overlay_manual_command): Ditto.
>  > > 	(overlay_off_command): Ditto.
>  > 
>  > Do any of our symtab maintainers have a comment on this patch?  I
>  > vaguely remember that you agreed with it on gdb@, Jim?
>  > 
> 
> I'll approve it.  I did some digging in the old Cygnus repository that
> dates back to 1991, and the code was already like that back then. So
> there really seem to be no rationale other than letting the user read
> the messages. Maybe back then nobody imagined that there would be more
> than a screen of output.

Yes, I did mean to approve it as part of the discussion on gdb@.


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