Re: [patch] Fixes problem setting breakpoint in dynamic loader

Daniel Jacobowitz
2006-07-06T02:57:18Z

Formatting notes.

On Wed, Jul 05, 2006 at 04:57:23PM -0700, PAUL GILLIAM wrote:
> + contain executable code and we can set a breakpoint there. */

Two spaces after period, here and elsewhere.

> + /* No symbol was found in a code section, so look elsewhere. */
> + for (bkpt_namep = solib_break_names; *bkpt_namep!=NULL; bkpt_namep++)

Line is too long; also related, spaces around operators.

> + /* On ABI's that use function descriptors that are in the data
> + section, */

Lost a bit of this comment?

--
Daniel Jacobowitz
CodeSourcery