This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
| Other format: | [Raw text] | |
On Friday 30 March 2012 12:16:52 nick clifton wrote:
> >> Write errors should either be handled or ignored, but aborting is not
> >> the right thing (it's not an internal error).
> >
> > so just do something like the following ?
> >
> > if (fwrite (str, 1, len, fp) != len)
> >
> > {
> >
> > /* Ignore. */
> >
> > }
>
> Yes. Although it would be simpler to just remove the whole if (...)
> statment block and just call fwrite without testing its return value.
>
> A patch modified in this way is approved.
the reason i didn't do that is it causes warnings since fwrite() is marked as
"warn on unused result"
-mike
Attachment:
signature.asc
Description: This is a digitally signed message part.
| Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
|---|---|---|
| Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |