Re: [PATCH 6/8] gdb/xcoffread: simplify xcoff_secnum_to_sections

Tom Tromey
2026-01-08T20:40:00Z

>>>>> "Simon" == Simon Marchi <simon.marchi@efficios.com> writes:

Simon> The only remaining caller of xcoff_secnum_to_sections doesn't care about
Simon> the section index in the objfile, so remove that part.

Simon> Return the BFD section as the return value of the function, instead of
Simon> an output parameter. Remove the unnecessary default in the switch, to
Simon> avoid a "jump to label over variable initialization" warning.

Simon> Rename to xcoff_secnum_to_section (singular).

Looks good.
Approved-By: Tom Tromey <tom@tromey.com>

Tom