[patch] Fixes problem setting breakpoint in dynamic loader

Daniel Jacobowitz drow@false.org
Thu Jul 6 02:57:00 GMT 2006


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



More information about the Gdb-patches mailing list