lwip-devel
[Top][All Lists]
Advanced

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

Re: [lwip-devel] [Patch] missing indent in stats.c


From: Kieran Mansley
Subject: Re: [lwip-devel] [Patch] missing indent in stats.c
Date: Tue, 6 Mar 2012 20:19:44 +0000

On 24 Feb 2012, at 20:46, Gisle Vanem wrote:

> Shouldn't there be some extra indenting in stats.c? Like this:
> 
> --- Git-latest\src\core\stats.c    Mon Oct 24 13:34:20 2011
> +++ src\core\stats.c Tue Nov 29 22:59:26 2011
> @@ -99,8 +99,8 @@
>  LWIP_PLATFORM_DIAG(("memerr: %"STAT_COUNTER_F"\n\t", igmp->memerr));
>  LWIP_PLATFORM_DIAG(("proterr: %"STAT_COUNTER_F"\n\t", igmp->proterr));
>  LWIP_PLATFORM_DIAG(("rx_v1: %"STAT_COUNTER_F"\n\t", igmp->rx_v1));
> -  LWIP_PLATFORM_DIAG(("rx_group: %"STAT_COUNTER_F"\n", igmp->rx_group));
> -  LWIP_PLATFORM_DIAG(("rx_general: %"STAT_COUNTER_F"\n", igmp->rx_general));
> +  LWIP_PLATFORM_DIAG(("rx_group: %"STAT_COUNTER_F"\n\t", igmp->rx_group));
> +  LWIP_PLATFORM_DIAG(("rx_general: %"STAT_COUNTER_F"\n\t", 
> igmp->rx_general));
>  LWIP_PLATFORM_DIAG(("rx_report: %"STAT_COUNTER_F"\n\t", igmp->rx_report));
>  LWIP_PLATFORM_DIAG(("tx_join: %"STAT_COUNTER_F"\n\t", igmp->tx_join));
>  LWIP_PLATFORM_DIAG(("tx_leave: %"STAT_COUNTER_F"\n\t", igmp->tx_leave));

It makes sense to me that they should all be consistent, and we need to add the 
missing \t that your patch adds.  Could you file a bug on savannah so that it 
doesn't get forgotten?

Thanks

Kieran


reply via email to

[Prev in Thread] Current Thread [Next in Thread]