fhandler_dev_floppy::lseek

[Corinna Vinschen]

On Wed, Sep 05, 2001 at 01:44:53PM +0400, egor duda wrote:
> Hi!
> 
> Tuesday, 04 September, 2001 Corinna Vinschen vinschen@redhat.com wrote:
> 
> CV> On 2001-02-28 you checked in a patch to fhandler_floppy.cc
> CV> (fhandler_dev_floppy::lseek) to determine the drive geometry.
> 
> CV> For some reason you added that using a `if (os_being_run == winNT)'
> CV> which in the meantime has been substituted by Chris by a `is (iswinnt)'.
> 
> CV> However, I don't understand why it's used at all. As far as I know
> CV> and from what I've read in MSDN, raw device access is not supported
> CV> on 9x/ME. So the above `if' wouldn't make any sense.
> 
> i'm  not sure w9x always perform appropriate parameter checking when
> driver ioctl is called. it may just crash. 
> 
> CV> The reason I'm asking is that I'm creating a OS capability class
> CV> to change all these if's to a more capability oriented view.
> CV> In the above case that could be handled by e.g.
> 
> CV>         if (wincap.can_determine_drive_geometry ())
> 
> CV> or something but if it's unneccessary I would better like to get
> CV> rid of this `if' at all.
> 
> well, i don't really know. the reason i've added this check was
> "better safe that sorry". maybe it's really unnecessary. i just don't
> fully trust MS in such matters.

Ok, thanks for that info. I will remove that check since 9x/ME will
already fail trying to open \\.\X:

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.