Re: Bug in linker for Win32.

[Ian Lance Taylor]

Eric Youngdale <address@hidden> writes:

>       It has been many years since I have submitted anything for binutils
> (1997 is the last ChangeLog entry that I see with my name on it), so my
> recollection of the preferred format is a bit hazy.

Good to see you back, in any case.

>       Does anyone know who handles gdb bug reports?  I sent something to
> bug-gdb, but that mailing list is choked with spam and I have no idea
> whether anyone monitors it or not.

See http://sourceware.org/gdb/mailing-lists/

> !             for (is2 = is;
> !                  is2 && is2->the_bfd->my_archive == arch;
> !                  is2 = (lang_input_statement_type *)is2->next)

I would say a space after the first ')'.

> +         char * pnt;

No space between '*' and 'p'.

> !             for (is3 = is;
> !                  is3 && is3->the_bfd->my_archive == arch;
> !                  is3 = (lang_input_statement_type *)is3->next)

Again a space after the first ')'.
>                 {
> !                     pnt = strrchr(is3->the_bfd->filename, '.');

Space before the '('.

> !                     if( pnt != NULL && strcmp(pnt, ".dll") != 0 )

Space before both '('.  No space after first ')'.  No space before
last ')'.  In other words
  if (pnt != NULL && strcmp (pnt, ".dll") != 0)

etc.

Ian