[ Charset UTF-8 unsupported, converting... ] > On Mon, May 7, 2018 at 10:26 AM, Conrad Meyer <c...@freebsd.org> wrote: > > > On Mon, May 7, 2018 at 8:45 AM, Rodney W. Grimes > > <free...@pdx.rh.cn85.dnsmgr.net> wrote: > > >> + * w...@tools.de (Wolfgang Solfrank, TooLs GmbH) +49-228-985800 > > > > > > If the original is by w...@tools.de, who is this version by? > > > Or is this simply copied from some other FreeBSD file? > > > > This is covered thoroughly in lines 1-10 of the commit email: > > > > > Author: mjg > > ... > > > memmove is repurposed bcopy (arguments swapped, return value added) > > > > It's also trivial to see from code comparison... It noticed it right away > while reviewing...
I believe both you and Conrad are placing to much weight on the commit message and no weight on the comment. Someone reading the code is not going to have the context we presently have. Comments in code need to stand alone, without any context of commit message creating them, or code else where, unless they directly reference that code else where. -- Rod Grimes rgri...@freebsd.org _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
