[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) c
From: |
Daniel Jacobowitz |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32 |
Date: |
Mon, 27 Jul 2009 09:42:08 -0400 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Mon, Jul 27, 2009 at 03:33:08PM +0200, Filip Navara wrote:
> On Mon, Jul 27, 2009 at 3:21 PM, Daniel Jacobowitz<address@hidden> wrote:
> > On Mon, Jul 27, 2009 at 10:02:04AM +0000, Filip Navara wrote:
> >> On Win32 the setvbuf function requires the last parameter to be size
> >> between 2 and INT_MAX bytes, so the calls always failed. Since the whole
> >> point of the calls is to set line-buffered mode for the file handle and
> >> that's not supported on Win32 anyway, conditionally remove them.
> >
> > Should they be unbuffered where _IOLBF is not supported, then? I
> > assume the line buffering was to make them more useful in event of a
> > crash.
>
> I guess your assumption is correct. I'd be fine with using _IONBF for
> the exec.c part, but setting it for stdout doesn't sound too useful to
> me.
I've found that leaving stdout buffered on Windows is a pain; on Unix
"isatty()" is reliable, but on Windows it breaks down at the
Cygwin/Windows interaction border. So your simulated programs end up
using block output all the time if you have a mingw32 qemu in a Cygwin
terminal, for instance.
--
Daniel Jacobowitz
CodeSourcery
- [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32, Filip Navara, 2009/07/27
- Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32, Daniel Jacobowitz, 2009/07/27
- Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32, Filip Navara, 2009/07/27
- Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32,
Daniel Jacobowitz <=
- Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32, Filip Navara, 2009/07/27
- Re: [Qemu-devel] [PATCH 3/5] Remove setvbuf(<handle>, NULL, _IOLBF, 0) calls for Win32, Daniel Jacobowitz, 2009/07/27