[RFA] clean-up of sym_read mainline flag (1/2)
Tristan Gingold
gingold@adacore.com
Wed Dec 2 11:37:00 GMT 2009
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.
More information about the Gdb-patches
mailing list