Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Nov 2012 17:57:04 -0500
From:      Ben Kaduk <minimarmot@gmail.com>
To:        Dimitry Andric <dim@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r242934 - head/sys/dev/xen/netback
Message-ID:  <CAK2BMK5c8wb9dSostjsjUHZOXGUG1KiWE1xHQi5sHtJMRMaSEQ@mail.gmail.com>
In-Reply-To: <201211122228.qACMSWbs020517@svn.freebsd.org>
References:  <201211122228.qACMSWbs020517@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Nov 12, 2012 at 5:28 PM, Dimitry Andric <dim@freebsd.org> wrote:
> Author: dim
> Date: Mon Nov 12 22:28:32 2012
> New Revision: 242934
> URL: http://svnweb.freebsd.org/changeset/base/242934
>
> Log:
>   Redo r242889, now using the method from projects/amd64_xen_pv r240747.
>
>   Reminded by:  kib
>   MFC after:    3 days
>
> Modified:
>   head/sys/dev/xen/netback/netback_unit_tests.c
>
> Modified: head/sys/dev/xen/netback/netback_unit_tests.c
> ==============================================================================
> --- head/sys/dev/xen/netback/netback_unit_tests.c       Mon Nov 12 22:01:29 2012        (r242933)
> +++ head/sys/dev/xen/netback/netback_unit_tests.c       Mon Nov 12 22:28:32 2012        (r242934)
> @@ -58,11 +58,11 @@ __FBSDID("$FreeBSD$");
>  #define        TOSTRING(x) STRINGIFY(x)
>
>  /**
> - * Writes an error message to buffer if cond is false, and returns true
> - * iff the assertion failed.  Note the implied parameters buffer and
> + * Writes an error message to buffer if cond is false
> + * Note the implied parameters buffer and

At risk of being Bruce, aren't comments like this supposed to be
complete sentences and end with a full stop?

-Ben Kaduk



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAK2BMK5c8wb9dSostjsjUHZOXGUG1KiWE1xHQi5sHtJMRMaSEQ>