Re: [PATCH spice-common] fix indentation on previous patch

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

 



Hi,

On Wed, May 31, 2017 at 03:13:13PM +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  common/log.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/common/log.c b/common/log.c
> index bab1afa..7437d3e 100644
> --- a/common/log.c
> +++ b/common/log.c
> @@ -156,7 +156,7 @@ static void spice_logv(const char *log_domain,
>      GLogLevelFlags glib_level = spice_log_level_to_glib(log_level);
>
>      if ((glib_level & G_LOG_LEVEL_MASK) > glib_debug_level) {
> -            return; // do not print anything
> +        return; // do not print anything
>      }

I find it fine for patches that are pending for some time to be reviewed
to be acked+pushed in short period of time.

Other than that a day or so as grace period would avoid this.
Just a small comment, not a big problem either way

Acked-by: Victor Toso <victortoso@xxxxxxxxxx>


>
>      log_msg = g_string_new(NULL);
> -- 
> 2.9.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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]