[RFA] Fix compilation warnings in irix-core.c
Eric Christopher
echristo@redhat.com
Tue Mar 29 06:29:00 GMT 2005
> > Spaces around the + please.
>
> Daniel and Thiemo suggested changing the + into a |. I will make that
> change and test that, if you agree.
>
I do. Don't forget the spacing :)
> > irix_core_core_file_matches_executable_p (core_bfd, exec_bfd)
> > - bfd *core_bfd, *exec_bfd;
> > + bfd *core_bfd ATTRIBUTE_UNUSED, *exec_bfd ATTRIBUTE_UNUSED;
> >
> > Can you move this to:
> >
> > bfd *core_bfd ATTRIBUTE_UNUSED;
> > bfd *exec_bfd ATTRIBUTE_UNUSED;
> >
> > please?
>
> Sure.
>
> > OK with those changes. Though if you want to migrate the whole file to
> > ISO I'd appreciate it. For style just look at, say, elfxx-mips.c :)
>
> Sure. I'll try to do that today.
>
Excellent.
-eric
More information about the Binutils
mailing list