Change looks good, but you are not consistent with your pragma removal... -Chris
Mark J. Nelson wrote: > This is a followup to my request for a closed code review, in which James > suggested Chris look at these changes. > > Once again, apologies to external folks on s-m-d. > > Open and closed webrevs at: > http://anthrax.central/net/mrliberal/export/ws/mjnelson/mercurial-push/webrev.scsi-failover-open/ > http://anthrax.central/net/mrliberal/export/ws/mjnelson/mercurial-push/usr/closed/webrev.scsi-failover-closed/ > > ...because this is coordinated, I won't push the change to our project > repository, but rather keep both open/closed changes in my > push-to-new-gate repositories, and fold 'em in with the recommit on > Tuesday. > > --Mark >