jiro@shaman.com (Jiro Nakamura) (03/06/91)
NeXT has recently contact me to say that the "bug" in scsireg.h was actually a feature. I mentioned in my previous posting that /usr/include/ nextdev/scsireg.h (on line 253, in System 2.0) read: char ir_revision[32]; /* revision level info in ascii */ My contention was that the SCSI standard only sets out 4 bytes for the revision data and that it must be a typo. NeXT's position is that in the standard, it does indeed set out 4 bytes for revision data, but the standard also makes the provision for additional information *after* the revision information. That is why they have made the last string array 32 bytes long. My position is that I understand that they are being helpful in providing additional buffer space in the structure (which I'm glad for, although other structures don't provide such "extensions"), but that it would be more intuitive if they had done it like: char ir_revision[4]; /* revision level info in ascii */ char additional_info[28]; /* Device specific information */ In the end, it might just be a matter of taste. Since I posted the original one as a bug, I was asked to post this article since it essentially is not a bug, but as I said, a matter of taste and programming style. - Jiro Nakamura jiro@shaman.com -- Jiro Nakamura jiro@shaman.com Shaman Consulting (607) 253-0687 VOICE "Bring your dead, dying shamans here!" (607) 253-7809 FAX/Modem