Tristan Gingold gingold@adacore.com
Wed Apr 14 09:30:00 GMT 2010

• Previous message (by thread): [Patch 2/4] Rewrite of the alpha-vms bfd back-end (bfd)
• Next message (by thread): Binutils documentation: info format
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

---

On Apr 13, 2010, at 1:12 PM, Nick Clifton wrote:

> Hi Tristan,

[...]
> 
> Approved - please apply.

Thanks, committed with the suggested improvements.

Tristan.

> 
> Some comments:
> 
> +/* Read & process EIHD record.
> +   Return 0 on success, -1 on error  */
> +
> +static int
> +_bfd_vms_slurp_eihd (bfd *abfd, unsigned int *eisd_offset,
> +                     unsigned int *eihs_offset)
> 
> Why not use bfd_boolean as the return type here ?  (And for other, similar functions ...)
> 
> 
> +	  sprintf (name, "$%s_%03d$", pfx, section_count++);
> 
> Is it (theoretically) possible to have more than 999 EISD records ?  If so then maybe an assertion or range check would be appropriate here.
> 
> 
> Cheers
>  Nick
> 

---

• Previous message (by thread): [Patch 2/4] Rewrite of the alpha-vms bfd back-end (bfd)
• Next message (by thread): Binutils documentation: info format
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

---

More information about the Binutils mailing list