Source-Changes-D archive

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

Re: CVS commit: src/sys/kern



In article <20131231171938.GA26368%netbsd.org@localhost>,
David Holland  <dholland-sourcechanges%netbsd.org@localhost> wrote:
>On Tue, Dec 24, 2013 at 09:47:04AM -0500, Christos Zoulas wrote:
> > Modified Files:
> >     src/sys/kern: kern_exec.c
> > 
> > Log Message:
> > replace strcpy with copystr and remove useless strcpy (Maxime Villard)
>
>...how is this a step forward?

The comment is enforced.

>(and that second strcpy is not useless, I put it there for a reason)

Can you explain the reason? It would be nice to have a comment about
things like this that are non-obvious.

christos



Home | Main Index | Thread Index | Old Index