[Patch] Fix type signature of feof and ferror in c++

[Corinna Vinschen]

On Dec 11 10:23, Jonathan Roelofs wrote:
> On 12/11/14 10:04 AM, Corinna Vinschen wrote:
> >On Dec 11 09:08, Jonathan Roelofs wrote:
> >>This fixes one of the libc++ tests: depr/depr.c.headers/stdio_h.pass.cpp
> >
> >>diff --git a/ChangeLog b/ChangeLog
> >>index 5bea065..640bccf 100644
> >>--- a/ChangeLog
> >>+++ b/ChangeLog
> >>@@ -1,3 +1,7 @@
> >>+2014-12-11  Jonathan Roelofs  <...>
> >>+
> >>+	* libc/include/stdio.h: Fix type signature of feof and ferror in c++
> >>+
> >
> >Please don't send ChangeLogs as diffs, always plain.  They won't apply
> >cleanly most of the time.
> Ok, yeah I could see that being annoying. Didn't know there was an etiquette
> for that. Should I be putting them in the body of the email when I send a
> patch?

Sure, that's absolutely ok.

> >Wouldn't it make more sense for all targets to move this into the
> >definitions of __sfeof/__sferror?
> Depends on what the type signature of __sfeof and __sferror are supposed to
> be... that I don't know.  If those are just internal newlib things, then I
> don't think it really matters, either way would work.

Ok, patch applied then.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20141211/5ec8f478/attachment.sig>