delorie.com/archives/browse.cgi | search |
Hi! Tuesday, 04 September, 2001 Corinna Vinschen vinschen AT redhat DOT 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. Egor. mailto:deo AT logos-m DOT ru ICQ 5165414 FidoNet 2:5020/496.19
webmaster | delorie software privacy |
Copyright © 2019 by DJ Delorie | Updated Jul 2019 |