[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