Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch/mips/mips



 On 11/09/2010 13:54, David Holland wrote:
On Tue, Nov 09, 2010 at 01:46:46PM +0200, Antti Kantee wrote:
  >  >  Thanks for the extra details.  It may be "obvious" with the background
  >  >  info you've now provided, but wasn't from your initial commit message.
  >
  >  Gotta love source-changes-full and nxr.netbsd.org ;)

No, it's not obvious from those either, and I'm even familiar with the
architecture. This whole thing could have been avoided if you'd
written a clear commit message; can you revise the text in CVS to have
fewer dangling referents?
Or better yet, improve the comments so people looking at the code know what the considerations are? Seems like an area where a sentence or two would save a lot of grief in the future...

Warner


Home | Main Index | Thread Index | Old Index