Re: More changes about open on Win95 directories.

2002-09-19 Thread Pierre A. Humblet
Christopher Faylor wrote: > > It really depends on the context. Some (e.g. fhandler_console) try to cover all bases but I don't think it's right, or at least necessary HANDLE h = CreateFileA ("CONIN$", GENERIC_READ, FILE_SHARE_WRITE, if (h == INVALID_HANDLE_VALUE || h == NULL) Pierre

Re: More changes about open on Win95 directories.

2002-09-19 Thread Christopher Faylor
On Thu, Sep 19, 2002 at 09:31:34AM +0100, Max Bowsher wrote: >Pierre A. Humblet wrote: >> Is '!' invalid? It can easily be confused with '|'. > >Maybe ':' ? Take a look at the context. >> I am bothered that the code uses 0 as an illegal >> handle value. Is that really the case? > >No. >/usr/incl

Re: More changes about open on Win95 directories.

2002-09-19 Thread Earnie Boyd
Chris January wrote: > > > > Is '!' invalid? It can easily be confused with '|'. > > > > Maybe ':' ? > > > > > I am bothered that the code uses 0 as an illegal > > > handle value. Is that really the case? > > > > No. > > /usr/include/w32api/winbase.h:232:#define INVALID_HANDLE_VALUE > > (HANDLE)(

RE: More changes about open on Win95 directories.

2002-09-19 Thread Chris January
> > Is '!' invalid? It can easily be confused with '|'. > > Maybe ':' ? > > > I am bothered that the code uses 0 as an illegal > > handle value. Is that really the case? > > No. > /usr/include/w32api/winbase.h:232:#define INVALID_HANDLE_VALUE > (HANDLE)(-1) It's not quite as simple as that... Alth

Re: More changes about open on Win95 directories.

2002-09-19 Thread Max Bowsher
Pierre A. Humblet wrote: > Is '!' invalid? It can easily be confused with '|'. Maybe ':' ? > I am bothered that the code uses 0 as an illegal > handle value. Is that really the case? No. /usr/include/w32api/winbase.h:232:#define INVALID_HANDLE_VALUE (HANDLE)(-1) Max.

Re: More changes about open on Win95 directories.

2002-09-18 Thread Christopher Faylor
On Thu, Sep 19, 2002 at 12:10:51AM -0400, Pierre A. Humblet wrote: >>>And yesterday's question: On line 173 of fhandler_disk_file.cc >>>[strpbrk (get_win32_name (), "?*|<>|")] is there a need for the >>>two '|'? Was something else meant? >> >>I removed it. I don't know if there is another inval

Re: More changes about open on Win95 directories.

2002-09-18 Thread Pierre A. Humblet
At 11:34 PM 9/18/2002 -0400, Christopher Faylor wrote: >> if (fl->l_len < 0) >>{ >> win32_start -= fl->l_len; >> win32_len = -fl->l_len; >>} >I've looked at that code a few times and wondered about that. It seems >backwards but maybe someone else has more insight. > Like, if st

Re: More changes about open on Win95 directories.

2002-09-18 Thread Christopher Faylor
On Wed, Sep 18, 2002 at 10:02:25PM -0400, Pierre A. Humblet wrote: >I fixed all that by adding set_nohandle () and get_nohandle () appropriately. >To avoid submitting patches on top of yesterday changes, I include cumulative >changes for the last two days. > >Also, on line 476 of fhandler_disk_fi

More changes about open on Win95 directories.

2002-09-18 Thread Pierre A. Humblet
Running more complicated tests around my changes of yesterday, I can produce the following system_printf by doing close on exec's and then forking: 121340 [main] a 86211543 fhandler_base::fork_fixup: /dev/zero - Win32 error 6, handle io_handle<0x0> 121

RE: open () on Win95 directories

2002-09-17 Thread Pierre A. Humblet
At 08:18 PM 9/17/2002 -0500, Gary R. Van Sickle wrote: >Hehehehe, I was just about to hit send on a reply to your original email on this >Pierre! It wasn't going to have a nice patch attached though. Thanks for >tracking this down and fixing it. This problem is fixed, but there is another one!

RE: open () on Win95 directories

2002-09-17 Thread Gary R. Van Sickle
] > [mailto:[EMAIL PROTECTED]]On Behalf Of Pierre A. Humblet > Sent: Tuesday, September 17, 2002 7:59 PM > To: [EMAIL PROTECTED] > Subject: open () on Win95 directories > > > As reported on Sunday in the Cygwin list, applications such as > mutt 1.4 do not work on Win95/98

open () on Win95 directories

2002-09-17 Thread Pierre A. Humblet
As reported on Sunday in the Cygwin list, applications such as mutt 1.4 do not work on Win95/98/ME because there Cygwin does not implement open() on directories, as CreateFile () does not work on directories either. However often open () is used on directories with fchdir () or fstat (), which