[Spice-devel] [PATCH 1/6] style: we are now using capital FIXME and TODO

Pavel Grunt pgrunt at redhat.com
Wed May 11 11:17:56 UTC 2016


Hi Frediano,

patches look good to me - ack.

Thanks,
Pavel

On Wed, 2016-05-11 at 10:42 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  docs/spice_style.txt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/spice_style.txt b/docs/spice_style.txt
> index 8966365..e4328fc 100644
> --- a/docs/spice_style.txt
> +++ b/docs/spice_style.txt
> @@ -59,10 +59,10 @@ Static storage initialization
>  
>  To improve code readability, explicitly initialize variables that depend on
> default initialization with zero/null.
>  
> -Fixme and todo
> +FIXME and TODO
>  --------------
>  
> -Comments that are prefixed with `fixme` describe a bug that need to be fixed.
> Generally, it is not allowed to commit new code having `fixme` comment.
> Committing  `fixme` is allowed only for existing bugs. Comments that are
> prefixed with `todo` describe further features, optimization or code
> improvements, and they are allowed to be committed along with the relevant
> code.
> +Comments that are prefixed with `FIXME` describe a bug that need to be fixed.
> Generally, it is not allowed to commit new code having `FIXME` comment.
> Committing  `FIXME` is allowed only for existing bugs. Comments that are
> prefixed with `TODO` describe further features, optimization or code
> improvements, and they are allowed to be committed along with the relevant
> code.
>  
>  ASSERT
>  ------


More information about the Spice-devel mailing list