Re: Changing the return value of xxx_attach() from void to int.

[David Holland <dholland-tech%netbsd.org@localhost>]

On Sat, Jul 09, 2016 at 04:57:20PM -0700, John Nemeth wrote:
 >      A number of people have expressed reservation (bring up memories
 > of device_t and how long that took to settle out) indicating that
 > this should be done on a branch or something.  Personally, I don't
 > see the need to do so.  The issue with the device_t change was that
 > it involved actual code changes.  This does not, it is simply search
 > and replace, which is a much less dangerous thing to do.  Some have
 > suggested doing other changes at the same time.  That would certainly
 > increase the risk.

The substance of that reservation is that there's not much point doing
it without also taking the time to correct the behavior, i.e., back
out properly if something fails. And that requires attention, not just
mechanical changes.

-- 
David A. Holland
dholland%netbsd.org@localhost