From owner-freebsd-scsi Fri May 31 06:25:32 1996 Return-Path: owner-freebsd-scsi Received: (from root@localhost) by freefall.freebsd.org (8.7.5/8.7.3) id GAA11622 for freebsd-scsi-outgoing; Fri, 31 May 1996 06:25:32 -0700 (PDT) Received: from Fieber-John.campusview.indiana.edu (Fieber-John.campusview.indiana.edu [149.159.1.34]) by freefall.freebsd.org (8.7.5/8.7.3) with ESMTP id GAA11617 for ; Fri, 31 May 1996 06:25:30 -0700 (PDT) Received: from localhost (jfieber@localhost) by Fieber-John.campusview.indiana.edu (8.7.5/8.7.3) with SMTP id IAA14329; Fri, 31 May 1996 08:25:22 -0500 (EST) X-Authentication-Warning: Fieber-John.campusview.indiana.edu: jfieber owned process doing -bs Date: Fri, 31 May 1996 08:25:21 -0500 (EST) From: John Fieber X-Sender: jfieber@Fieber-John.campusview.indiana.edu To: hdalog@zipnet.net cc: scsi@FreeBSD.org Subject: Re: Archive Viper not correctly identified. In-Reply-To: <199605311200.IAA06323@hda> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-scsi@FreeBSD.org X-Loop: FreeBSD.org Precedence: bulk On Fri, 31 May 1996, Peter Dufault wrote: > (add a "*" to the archive list) > > Right now a short name has implied spaces at the end - does an implied > "*" at the end of a short name makes more sense? Having an explicit wildcard there makes it reasonably clear to someone browsing the code how the matching is intended to work without having to read the actual matching code. Also, unless an explicit space is added after a name, you could get a false match. Imagine archive makes a viper 1500. The string "VIPER 150" would match "VIPER 1500" which may not be desired. I think requiring a * to ensure a match will be more intuitive and less prone to programmer error than requiring a trailing space to prevent a false match. (not that it would occur often, but it might...) Regardless, a more informative comment at the head of the quirk list describing the policy, and mentioning the fact that the manufacturer, model and revision fields are 8, 16 and 4 bytes respectively would be helpful to future quriklist maintainers. -john == jfieber@indiana.edu =========================================== == http://fallout.campusview.indiana.edu/~jfieber ================