Subject: CVS commit: syssrc
To: None <source-changes@netbsd.org>
From: Matthew Jacob <mjacob@netbsd.org>
List: source-changes
Date: 12/30/2000 21:47:21
Module Name:	syssrc
Committed By:	mjacob
Date:		Sat Dec 30 19:47:21 UTC 2000

Modified Files:
	syssrc/sys/dev/ic: isp.c

Log Message:
[ dummy checkin to add to previous CVS ]

It's probably not really a compiler bug- somebody pointed out
that it was the kernel making strings readonly. But I do think it's
a bug. The actual code was really more like:

	char *revname;

	...

	revname = "2X00";

	...
	revname[1] = '2'; <<<<<<<<< BOOOM

The variable revname is not a const. If I had said

	const char *revname = "2X00"
	...
	revname[1] = '2';

that would indeed be breaking const rules.


To generate a diff of this commit:
cvs rdiff -r1.67 -r1.68 syssrc/sys/dev/ic/isp.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.