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

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

 



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@xxxxxxxxxx>
> ---
>  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
>  ------
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]