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: [RFA] clean-up of sym_read mainline flag (1/2)


On Nov 30, 2009, at 9:19 PM, Tom Tromey wrote:

>>>>>> "Tristan" == Tristan Gingold <gingold@adacore.com> writes:
> 
> Tristan> The mainline flag from sym_read routines was passed as argument
> Tristan> to functions that doesn't use it.  To clean-up a little bit and
> Tristan> to make the code more readable, this patch removes this unused
> Tristan> parameter.
> 
> This is almost ok.
> 
> Tristan>  void
> Tristan> -coffstab_build_psymtabs (struct objfile *objfile, int mainline,
> Tristan> +coffstab_build_psymtabs (struct objfile *objfile,
> Tristan>  			 CORE_ADDR textaddr, unsigned int textsize,
> 
> I spot-checked and noticed that this function (and some others in this
> file) documents the 'mainline' parameter in its header comment.  These
> comments must also be updated.
> 
> This is ok with that change.  Please be sure to check all the functions,

I have updated the comments too, and I hope I checked all the functions.

Thanks,
Tristan.


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