Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 23 Feb 2017 18:32:18 +0100
From:      Mateusz Guzik <mjguzik@gmail.com>
To:        "Pedro F. Giffuni" <pfg@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r314142 - head/sys/dev/ce
Message-ID:  <20170223173218.GB5744@dft-labs.eu>
In-Reply-To: <201702231530.v1NFULwx024028@repo.freebsd.org>
References:  <201702231530.v1NFULwx024028@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Feb 23, 2017 at 03:30:21PM +0000, Pedro F. Giffuni wrote:
> Author: pfg
> Date: Thu Feb 23 15:30:21 2017
> New Revision: 314142
> URL: https://svnweb.freebsd.org/changeset/base/314142
> 
> Log:
>   dev/ce: double assignment.
>   
>   The code is not operational right now so just comment away an obviously
>   useless assignment. Fix some typos while here.
>   
>  
> -	/* XXXRIK: This function should be for comleteness, but for now I
> -	 * don't use it. So I just start to write and didn't finished it yet.
> -	 * It and it is VERY BUGGY!!! Do not use it. If you really
> -	 * need it ask me to fix it or rewrite it by your self.
> +	/* XXXRIK: This function should be for completeness, but for now I
> +	 * don't use it. I just started to write and haven't finished it yet.
> +	 * It is VERY BUGGY!!! Do not use it. If you really need
> +	 * it ask me to fix it or rewrite it by yourself.
>  	 * Note: most buggy part of it in ce_on_config_stop!
>  	 */

Please don't fix typos etc. unless they seriouslyl harm readability.
Changes of this sort make it harder to svn blame.

-- 
Mateusz Guzik <mjguzik gmail.com>



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