cvs commit: src/usr.bin/mt mt.c

[Maxime Henrion]

Ruslan Ermilov wrote:
> On Fri, Apr 09, 2004 at 07:19:27AM -0700, Maxime Henrion wrote:
> > mux         2004/04/09 07:19:12 PDT
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     usr.bin/mt           mt.c 
> >   Log:
> >   Reintroduce a comment that was wrongly removed in revision 1.34.
> >   
> >   Revision  Changes    Path
> >   1.35      +4 -0      src/usr.bin/mt/mt.c
> > 
> Err, the weirdness was that different drivers returned different MT_*'s:
> 
> -       /*
> -        * XXX This is weird.  The st driver reports the tape drive
> -        * as 0x7 (MT_ISAR - Sun/Archive compatible); the wt driver
> -        * either reports MT_ISVIPER1 for an Archive tape, or 0x11
> -        * (MT_ISMFOUR) for other tapes.
> -        * XXX for the wt driver, rely on it behaving like a "standard"
> -        * magtape driver.
> 
> > Index: src/usr.bin/mt/mt.c
> > diff -u src/usr.bin/mt/mt.c:1.34 src/usr.bin/mt/mt.c:1.35
> > --- src/usr.bin/mt/mt.c:1.34	Sun Mar 14 04:09:43 2004
> > +++ src/usr.bin/mt/mt.c	Fri Apr  9 07:19:12 2004
> > @@ -44,7 +44,7 @@
> >  #endif /* not lint */
> >  
> >  #include <sys/cdefs.h>
> > -__FBSDID("$FreeBSD: /repoman/r/ncvs/src/usr.bin/mt/mt.c,v 1.34 2004/03/14 12:09:43 ru Exp $");
> > +__FBSDID("$FreeBSD: /repoman/r/ncvs/src/usr.bin/mt/mt.c,v 1.35 2004/04/09 14:19:12 mux Exp $");
> >  
> >  /*
> >   * mt --
> > @@ -312,6 +312,10 @@
> >  	char	*t_dsbits;	/* "drive status" register */
> >  	char	*t_erbits;	/* "error" register */
> >  } tapes[] = {
> > +	/*
> > +	 * XXX This is weird.  The st driver reports the tape drive
> > +	 * as 0x7 (MT_ISAR - Sun/Archive compatible).
> > +	 */
> >  	{ MT_ISAR,	"SCSI tape drive", 0,		0 },
> >  	{ 0, NULL, 0, 0 }
> >  };
> 
> I don't see how this comment applies today and in its current shape.

Hmmm, the wording was unclear to me and I'm still not sure about this.
However, I assume you have much better english skills than me and I'll
thus back it out.

> In any case, I'd appreciate it if you notified me about what I did
> wrong before committing.

Come on, don't get overly offended for that, this is silly.  We all fix
bugs continuously, and if we had to mail the person who is responsible
for every bug we fix, we wouldn't have much time left for coding...
I could have agreed if this was code, but it's a simple comment change.

Cheers,
Maxime