Re: CVS commit: src/lib/libc/stdio

2021-02-01 Thread Robert Elz
Date:Mon, 1 Feb 2021 17:50:54 + From:"Jaromir Dolecek" Message-ID: <20210201175054.112e7f...@cvs.netbsd.org> | FreeBSD has a similar check, but they return EINVAL instead, feel | free to adjust if SUS or other standard mandates specific value Not currently

Re: CVS commit: src/lib/libc/stdio

2021-01-31 Thread Robert Elz
Date:Sun, 31 Jan 2021 18:34:22 +0100 From:Joerg Sonnenberger Message-ID: | That makes no sense. Just turn them into a short read, which is | something users have to deal with anyway. I'm not sure I agree with that one. If the user's size * nmemb overflows a

Re: CVS commit: src/lib/libc/stdio

2021-01-31 Thread Kamil Rytarowski
On 31.01.2021 18:34, Joerg Sonnenberger wrote: > On Sun, Jan 31, 2021 at 05:27:28PM +0100, Kamil Rytarowski wrote: >> On 31.01.2021 17:18, Jaromir Dolecek wrote: >>> Module Name:src >>> Committed By: jdolecek >>> Date: Sun Jan 31 16:18:22 UTC 2021 >>> >>> Modified

Re: CVS commit: src/lib/libc/stdio

2021-01-31 Thread Joerg Sonnenberger
On Sun, Jan 31, 2021 at 05:27:28PM +0100, Kamil Rytarowski wrote: > On 31.01.2021 17:18, Jaromir Dolecek wrote: > > Module Name:src > > Committed By: jdolecek > > Date: Sun Jan 31 16:18:22 UTC 2021 > > > > Modified Files: > > src/lib/libc/stdio: fread.c > > > >

Re: CVS commit: src/lib/libc/stdio

2021-01-31 Thread Kamil Rytarowski
On 31.01.2021 17:18, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Sun Jan 31 16:18:22 UTC 2021 > > Modified Files: > src/lib/libc/stdio: fread.c > > Log Message: > for unbuffered I/O arrange for the destination buffer to be filled in one > go, instead

Re: CVS commit: src/lib/libc/stdio

2014-09-30 Thread Martin Husemann
On Mon, Sep 29, 2014 at 09:35:04PM +0200, Steffen Nurpmeso wrote: Not being able to get to the list of pull-ups from that page, or at least getting a hint of where to find them is... a mistake. I wouldn't call it a mistake (though links certainly can be added) - that page documents something

Re: CVS commit: src/lib/libc/stdio

2014-09-30 Thread Steffen Nurpmeso
Martin Husemann mar...@duskware.de wrote: |On Mon, Sep 29, 2014 at 09:35:04PM +0200, Steffen Nurpmeso wrote: | Not being able to get to the list of pull-ups from that page, or | at least getting a hint of where to find them is... a mistake. | |I wouldn't call it a mistake (though links

Re: CVS commit: src/lib/libc/stdio

2014-09-29 Thread Martin Husemann
On Mon, Sep 29, 2014 at 08:29:10PM +0200, Steffen Nurpmeso wrote: (And also i would place a link to the current pull-ups (to the wiki) on www.netbsd.org/developers/releng/pullups.html, since Google shows the latter first, yet that is astonishing empty.) Can you say that again, please? Martin

Re: CVS commit: src/lib/libc/stdio

2014-09-29 Thread Steffen Nurpmeso
|Module Name: src |Committed By: christos |Date: Mon Sep 29 14:58:33 UTC 2014 | |Modified Files: | src/lib/libc/stdio: printf.3 vsnprintf.c vsnprintf_ss.c | |Log Message: |Return EOVERFLOW like FreeBSD does if the buffer size exceeds INT_MAX |(well FreeBSD documents INT_MAX + 1,

Re: CVS commit: src/lib/libc/stdio

2014-09-29 Thread Alan Barrett
On Mon, 29 Sep 2014, Martin Husemann wrote: On Mon, Sep 29, 2014 at 08:29:10PM +0200, Steffen Nurpmeso wrote: (And also i would place a link to the current pull-ups (to the wiki) on www.netbsd.org/developers/releng/pullups.html, since Google shows the latter first, yet that is astonishing

Re: CVS commit: src/lib/libc/stdio

2014-09-29 Thread Steffen Nurpmeso
Martin Husemann mar...@duskware.de wrote: |On Mon, Sep 29, 2014 at 08:29:10PM +0200, Steffen Nurpmeso wrote: | (And also i would place a link to the current pull-ups (to the | wiki) on www.netbsd.org/developers/releng/pullups.html, since | Google shows the latter first, yet that is astonishing

Re: CVS commit: src/lib/libc/stdio

2012-03-27 Thread Takehiko NOZAKI
Hi, It seems that lint(1) is not cross build safe, it doesn't handle MD char default type of sign/unsignd. See src/usr.bin/xlint/lint1/tree.c::cvtcon(). They use host MD CHAR_MAX directry ;) So, if cross building ppc/arm on other arch cause false alarm , out of range warnng. Regards. --

Re: CVS commit: src/lib/libc/stdio

2012-03-27 Thread Valeriy E. Ushakov
On Tue, Mar 27, 2012 at 20:19:41 +, Christos Zoulas wrote: In article CAN6pqGQMV-JtfuidyRAH1VoYV7mJott=aruq3t3nro1kwcd...@mail.gmail.com, Takehiko NOZAKI takehiko.noz...@gmail.com wrote: -=-=-=-=-=- Hi, It seems that lint(1) is not cross build safe, it doesn't handle MD char

Re: CVS commit: src/lib/libc/stdio

2012-03-27 Thread Christos Zoulas
In article 20120327202907.gt26...@bigmac.stderr.spb.ru, Valeriy E. Ushakov u...@stderr.spb.ru wrote: But that is not what the code was. The code was: char c; if (c == CHAR_MAX) ... and *that* is portable. As I said in another mail to thsi thread that went unanswered, it is literally

Re: CVS commit: src/lib/libc/stdio

2012-03-27 Thread Cherry G. Mathew
On 28 March 2012 07:53, Christos Zoulas chris...@astron.com wrote: In article 20120327202907.gt26...@bigmac.stderr.spb.ru, Valeriy E. Ushakov u...@stderr.spb.ru wrote: But that is not what the code was.  The code was:    char c; if (c == CHAR_MAX) ... and *that* is portable.  As I said in

Re: CVS commit: src/lib/libc/stdio

2012-03-21 Thread Christos Zoulas
In article 20120322003141.gb1...@britannica.bec.de, Joerg Sonnenberger jo...@britannica.bec.de wrote: On Wed, Mar 21, 2012 at 10:20:47AM -0400, Christos Zoulas wrote: Module Name: src Committed By:christos Date:Wed Mar 21 14:20:47 UTC 2012 Modified Files:

Re: CVS commit: src/lib/libc/stdio

2012-03-21 Thread Joerg Sonnenberger
On Thu, Mar 22, 2012 at 01:31:41AM +, Christos Zoulas wrote: In article 20120322003141.gb1...@britannica.bec.de, Joerg Sonnenberger jo...@britannica.bec.de wrote: On Wed, Mar 21, 2012 at 10:20:47AM -0400, Christos Zoulas wrote: Module Name: src Committed By: christos Date:

Re: CVS commit: src/lib/libc/stdio

2012-03-21 Thread Takehiko NOZAKI
Hi, It seems that lint(1) is not cross build safe, it doesn't handle MD char default type of sign/unsignd. See src/usr.bin/xlint/lint1/tree.c::cvtcon(). They use host MD CHAR_MAX directry ;) So, if cross building ppc/arm on other arch cause false alarm , out of range warnng. very truly yours.

Re: CVS commit: src/lib/libc/stdio

2012-02-17 Thread NONAKA Kimihiro
Hi, 2012/2/18 Christos Zoulas chris...@netbsd.org: Module Name:    src Committed By:   christos Date:           Fri Feb 17 19:57:53 UTC 2012 Modified Files:        src/lib/libc/stdio: vfwprintf.c Log Message: Fix: CVE-2012-0864 fprintf() positional argument abuse. Described in:

Re: CVS commit: src/lib/libc/stdio

2011-09-11 Thread Jukka Ruohonen
On Mon, Sep 12, 2011 at 02:41:07AM +, David Holland wrote: On Sun, Sep 11, 2011 at 07:37:06AM +, Jukka Ruohonen wrote: Modified Files: src/lib/libc/stdio: funopen.3 Log Message: It is not just funopen(3) that is a BSDism. While that's true, those functions aren't

Re: CVS commit: src/lib/libc/stdio

2010-10-24 Thread Robert Elz
Date:Sun, 24 Oct 2010 16:00:46 +0100 From:Matthias Scheler t...@netbsd.org Message-ID: 20101024150046.ga...@colwyn.zhadum.org.uk | It doesn't have to be a macro anyway as it used only ones. Yes, I meant to ask about that, simply written inline would be no problem.

Re: CVS commit: src/lib/libc/stdio

2010-10-24 Thread Matthias Scheler
On Mon, Oct 25, 2010 at 12:24:14AM +0700, Robert Elz wrote: Date:Sun, 24 Oct 2010 16:00:46 +0100 From:Matthias Scheler t...@netbsd.org Message-ID: 20101024150046.ga...@colwyn.zhadum.org.uk | It doesn't have to be a macro anyway as it used only ones. Yes, I

Re: CVS commit: src/lib/libc/stdio

2010-10-23 Thread Jukka Ruohonen
On Fri, Oct 22, 2010 at 05:29:46PM -0400, Christos Zoulas wrote: Module Name: src Committed By: christos Date: Fri Oct 22 21:29:46 UTC 2010 Modified Files: src/lib/libc/stdio: ftell.c local.h Log Message: implement EOVERFLOW To generate a diff of this commit: cvs

Re: CVS commit: src/lib/libc/stdio

2010-10-23 Thread Matthias Scheler
On Sat, Oct 23, 2010 at 10:12:51AM -0400, Christos Zoulas wrote: Module Name: src Committed By: christos Date: Sat Oct 23 14:12:51 UTC 2010 Modified Files: src/lib/libc/stdio: local.h Log Message: tell lint to shut up. I'm sorry but this whole code is horrible. What is

Re: CVS commit: src/lib/libc/stdio

2010-09-28 Thread Alistair Crooks
On Tue, Sep 28, 2010 at 11:06:39AM +0900, Masao Uebayashi wrote: Index: src/lib/libc/stdio/fmemopen.c diff -u src/lib/libc/stdio/fmemopen.c:1.3 src/lib/libc/stdio/fmemopen.c:1.4 --- src/lib/libc/stdio/fmemopen.c:1.3 Sat Sep 25 14:00:30 2010 +++ src/lib/libc/stdio/fmemopen.c Mon Sep

Re: CVS commit: src/lib/libc/stdio

2010-09-28 Thread Masao Uebayashi
On Tue, Sep 28, 2010 at 08:28:04AM +0200, Alistair Crooks wrote: On Tue, Sep 28, 2010 at 11:06:39AM +0900, Masao Uebayashi wrote: Index: src/lib/libc/stdio/fmemopen.c diff -u src/lib/libc/stdio/fmemopen.c:1.3 src/lib/libc/stdio/fmemopen.c:1.4 --- src/lib/libc/stdio/fmemopen.c:1.3

Re: CVS commit: src/lib/libc/stdio

2010-09-27 Thread Masao Uebayashi
Index: src/lib/libc/stdio/fmemopen.c diff -u src/lib/libc/stdio/fmemopen.c:1.3 src/lib/libc/stdio/fmemopen.c:1.4 --- src/lib/libc/stdio/fmemopen.c:1.3 Sat Sep 25 14:00:30 2010 +++ src/lib/libc/stdio/fmemopen.c Mon Sep 27 16:50:13 2010 @@ -79,16 +79,18 @@ if (p-cur = p-tail)

Re: CVS commit: src/lib/libc/stdio

2010-05-15 Thread David Holland
On Fri, May 14, 2010 at 08:29:34AM +0300, Jukka Ruohonen wrote: Yes. The trouble is that because libraries and headers don't match up, you need to parse headers to extract the lists, and that seems likely to end up being delicate and a constant source of annoyance. How about the

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Thu, May 06, 2010 at 03:17:38PM +0300, Jukka Ruohonen wrote: Perhaps we need a bot to watch all commits and send reminders like You added a new file, but you haven't updated the sets, see URL for instructions, You added a new sysctl node but you haven't updated sysctl(7), etc.

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Fri, May 14, 2010 at 05:10:24AM +, David Holland wrote: Anyway, can you (at your convenience) send me a list of the things like this that need consistency checking? I will then have a shot at writing some scripts and we can see what things look like. er wait, there's a list upthread.

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread David Holland
On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to

Re: CVS commit: src/lib/libc/stdio

2010-05-13 Thread Jukka Ruohonen
On Fri, May 14, 2010 at 05:10:24AM +, David Holland wrote: Yes. The trouble is that because libraries and headers don't match up, you need to parse headers to extract the lists, and that seems likely to end up being delicate and a constant source of annoyance. How about the section 4

Re: CVS commit: src/lib/libc/stdio

2010-05-07 Thread Alan Barrett
On Thu, 06 May 2010, Christos Zoulas wrote: The fileno() function may fail if: [EBADF] The stream argument is not a valid stream, or the stream is not associated with a file. Isn't the above EBADF exactly what you are after? Our stdio supports funopen() which has

Re: CVS commit: src/lib/libc/stdio

2010-05-07 Thread Christos Zoulas
In article 20100507063258.ga23...@apb-laptoy.apb.alt.za, Alan Barrett a...@cequrux.com wrote: On Thu, 06 May 2010, Christos Zoulas wrote: The fileno() function may fail if: [EBADF] The stream argument is not a valid stream, or the stream is not associated with a file.

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread David Holland
On Wed, May 05, 2010 at 08:05:20AM +0200, Alan Barrett wrote: On Wed, 05 May 2010, Jukka Ruohonen wrote: Few examples of pages that are badly and *systematically* behind: intro(3) intro(4) pci(4) usb(4) mk.conf(5) sysctl(7) Perhaps we need a bot to

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread David Holland
On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to EBADF, hence diverging from the standard in this small detail. How is that not

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Jukka Ruohonen
On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to EBADF,

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Jukka Ruohonen
On Thu, May 06, 2010 at 12:04:53PM +0300, Jukka Ruohonen wrote: On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Jukka Ruohonen
On Thu, May 06, 2010 at 07:48:59AM +, David Holland wrote: On Wed, May 05, 2010 at 08:05:20AM +0200, Alan Barrett wrote: Perhaps we need a bot to watch all commits and send reminders like You added a new file, but you haven't updated the sets, see URL for instructions, You added a

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Joerg Sonnenberger
On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in such cases the NetBSD implementation does not set errno to EBADF,

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Jukka Ruohonen
On Thu, May 06, 2010 at 02:48:32PM +0200, Joerg Sonnenberger wrote: On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about return values: fileno() may fail and return -1. Note that in

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Joerg Sonnenberger
On Thu, May 06, 2010 at 06:00:45PM +0300, Jukka Ruohonen wrote: On Thu, May 06, 2010 at 02:48:32PM +0200, Joerg Sonnenberger wrote: On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the discussion about

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Matthias Scheler
On 6 May 2010, at 9:14, Jukka Ruohonen wrote: Module Name: src Committed By: jruoho Date: Thu May 6 08:14:08 UTC 2010 Modified Files: src/lib/libc/stdio: ferror.3 Log Message: Correct the discussion about return values: fileno() may fail and return -1. Note that in

Re: CVS commit: src/lib/libc/stdio

2010-05-06 Thread Christos Zoulas
In article 20100506150045.ga...@marx.bitnet, Jukka Ruohonen jruoho...@iki.fi wrote: On Thu, May 06, 2010 at 02:48:32PM +0200, Joerg Sonnenberger wrote: On Thu, May 06, 2010 at 08:51:29AM +, David Holland wrote: On Thu, May 06, 2010 at 08:14:08AM +, Jukka Ruohonen wrote: Correct the

Re: CVS commit: src/lib/libc/stdio

2010-05-05 Thread Alan Barrett
On Wed, 05 May 2010, Jukka Ruohonen wrote: Few examples of pages that are badly and *systematically* behind: intro(3) intro(4) pci(4) usb(4) mk.conf(5) sysctl(7) Perhaps we need a bot to watch all commits and send reminders like You added a new file,

Re: CVS commit: src/lib/libc/stdio

2010-05-05 Thread Mindaugas Rasiukevicius
Jukka Ruohonen jruoho...@iki.fi wrote: On Wed, May 05, 2010 at 12:58:13AM +0700, Robert Elz wrote: | As something like gets() has been standardized for ages, it makes | sense to explicitly note that this may no longer be true (with | respect to POSIX). That's where I disagree, it

Re: CVS commit: src/lib/libc/stdio

2010-05-05 Thread Jukka Ruohonen
On Wed, May 05, 2010 at 08:52:23AM +0100, Mindaugas Rasiukevicius wrote: History is indeed important, however it belongs to books, articles, etc. Other part of history belongs to version control logs, as Robert already mentioned. If I will want to figure out why and by whom some function was

Re: CVS commit: src/lib/libc/stdio

2010-05-05 Thread Joerg Sonnenberger
On Wed, May 05, 2010 at 06:53:06AM +0300, Jukka Ruohonen wrote: Please revert this change. I will rever it. Can you change it to use .Xr please? That makes proper cross-references in HTML output possible. Joerg

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Robert Elz
Date:Fri, 30 Apr 2010 06:00:14 + From:Jukka Ruohonen jru...@netbsd.org Message-ID: 20100430060014.7739117...@cvs.netbsd.org | Module Name:src | Committed By: jruoho | Date: Fri Apr 30 06:00:14 UTC 2010 | | Modified Files: |

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Jukka Ruohonen
On Tue, May 04, 2010 at 06:35:48PM +0700, Robert Elz wrote: I don't understand the change - that is, I don't understand the sentence that was added to the man page, it says ... The revision marked gets() as obsolete... What revision??? In my manual page: The functions fgets() and

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Joerg Sonnenberger
On Tue, May 04, 2010 at 04:39:16PM +0300, Jukka Ruohonen wrote: In my manual page: The functions fgets() and gets() conform to ANSI X3.159-1989 (``ANSI C89'') and IEEE Std 1003.1-2001 (``POSIX.1''). The IEEE Std 1003.1-2008 (``POSIX.1'') revision marked gets() as obsolescent,

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Robert Elz
Date:Tue, 4 May 2010 16:39:16 +0300 From:Jukka Ruohonen jruoho...@iki.fi Message-ID: 20100504133916.ga9...@marx.bitnet | In my manual page: [...] | What was the problem again? Something odd - sorry - it looks as if my (older) nroff -mandoc on current sources

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Jukka Ruohonen
On Wed, May 05, 2010 at 12:58:13AM +0700, Robert Elz wrote: | As something like gets() has been standardized for ages, it makes sense to | explicitly note that this may no longer be true (with respect to POSIX). That's where I disagree, it is just bloat - once it stops being a

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Luke Mewburn
On Tue, May 04, 2010 at 07:43:12AM +, Jukka Ruohonen wrote: | Module Name:src | Committed By: jruoho | Date: Tue May 4 07:43:12 UTC 2010 | | Modified Files: | src/lib/libc/stdio: stdio.3 | | Log Message: | Remove the list of functions. |

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Jukka Ruohonen
On Wed, May 05, 2010 at 01:03:49PM +1000, Luke Mewburn wrote: On Tue, May 04, 2010 at 07:43:12AM +, Jukka Ruohonen wrote: | Remove the list of functions. | | This list was updated only two times in nearly two decades. | | (If people need to learn the standard I/O functions in

Re: CVS commit: src/lib/libc/stdio

2010-05-04 Thread Jukka Ruohonen
On Wed, May 05, 2010 at 01:03:49PM +1000, Luke Mewburn wrote: I disagree; a list of all the public APIs for a given library in the main man page for that library is very useful. One more note I have made while skimming through the manual pages. Few examples of pages that are badly and

Re: CVS commit: src/lib/libc/stdio

2009-10-26 Thread David Holland
On Sun, Oct 25, 2009 at 05:09:34PM +, David Laight wrote: Lint is differentially far too picky... Remove some warnings that only appear on i386 (not on amd64) and that for some reason best known to others are deemed fatal for i386. Making this code 'pass lint' does absolutely nothing

Re: CVS commit: src/lib/libc/stdio

2009-10-26 Thread Matthias Scheler
On Mon, Oct 26, 2009 at 06:14:36AM +, David Holland wrote: On Sun, Oct 25, 2009 at 05:09:34PM +, David Laight wrote: Lint is differentially far too picky... Remove some warnings that only appear on i386 (not on amd64) and that for some reason best known to others are deemed fatal

Re: CVS commit: src/lib/libc/stdio

2009-10-26 Thread David Laight
On Sun, Oct 25, 2009 at 05:42:20PM +, Christos Zoulas wrote: Can we just revert the past 2 commits? Changing: (size_t)x - x + 0u does not look like an improvement to me. At least the first shows the intent, the second is just confusing, specially when size_t is unsigned long.

Re: CVS commit: src/lib/libc/stdio

2009-10-26 Thread James Chacon
On Oct 26, 2009, at 4:02 PM, David Laight wrote: On Sun, Oct 25, 2009 at 05:42:20PM +, Christos Zoulas wrote: Can we just revert the past 2 commits? Changing: (size_t)x - x + 0u does not look like an improvement to me. At least the first shows the intent, the second is just

Re: CVS commit: src/lib/libc/stdio

2009-10-25 Thread Christos Zoulas
In article 20091025170934.bd83c17...@cvs.netbsd.org, David Laight source-changes-d@NetBSD.org wrote: -=-=-=-=-=- Module Name: src Committed By: dsl Date: Sun Oct 25 17:09:34 UTC 2009 Modified Files: src/lib/libc/stdio: asprintf.c fgets.c fgetwc.c fread.c fvwrite.c

Re: CVS commit: src/lib/libc/stdio

2009-10-15 Thread David Laight
On Thu, Oct 15, 2009 at 12:01:01AM +0200, Alan Barrett wrote: On Wed, 14 Oct 2009, David Laight wrote: Module Name:src Committed By: dsl Date: Wed Oct 14 21:25:52 UTC 2009 Modified Files: src/lib/libc/stdio: fgets.c vfprintf.c Log Message:

Re: CVS commit: src/lib/libc/stdio

2009-10-14 Thread Alan Barrett
On Wed, 14 Oct 2009, David Laight wrote: Module Name: src Committed By: dsl Date: Wed Oct 14 21:25:52 UTC 2009 Modified Files: src/lib/libc/stdio: fgets.c vfprintf.c Log Message: Change a while () {} into a do {} while() so that fgets(buf, 1, file) detects EOF on an