[ovs-dev] [PATCH] lib/rstp-common: Remove double spaces.

Jarno Rajahalme jrajahalme at nicira.com
Wed Sep 10 16:49:24 UTC 2014


I personally prefer starting the struct field names on the same column whenever that is practical. However, this is better than the original inconsistency in style, so:

Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>

Pushed to master,

  Jarno

On Sep 10, 2014, at 1:28 AM, Daniele Venturino <daniele.venturino at m3s.it> wrote:

> Signed-off by: Daniele Venturino <daniele.venturino at m3s.it>
> 
> ---
> lib/rstp-common.h | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/lib/rstp-common.h b/lib/rstp-common.h
> index 4e0d398..587f88c 100644
> --- a/lib/rstp-common.h
> +++ b/lib/rstp-common.h
> @@ -225,19 +225,19 @@ enum rstp_bpdu_flag {
> OVS_PACKED(
> struct rstp_bpdu {
>     ovs_be16 protocol_identifier;
> -    uint8_t  protocol_version_identifier;
> -    uint8_t  bpdu_type;
> -    uint8_t  flags;
> +    uint8_t protocol_version_identifier;
> +    uint8_t bpdu_type;
> +    uint8_t flags;
>     ovs_be64 root_bridge_id;
>     ovs_be32 root_path_cost;
>     ovs_be64 designated_bridge_id;
>     ovs_be16 designated_port_id;
> -    ovs_be16  message_age;
> -    ovs_be16  max_age;
> -    ovs_be16  hello_time;
> -    ovs_be16  forward_delay;
> -    uint8_t  version1_length;
> -    uint8_t  padding[7];
> +    ovs_be16 message_age;
> +    ovs_be16 max_age;
> +    ovs_be16 hello_time;
> +    ovs_be16 forward_delay;
> +    uint8_t version1_length;
> +    uint8_t padding[7];
> });
> 
> enum rstp_admin_point_to_point_mac_state {
> -- 
> 1.8.1.2
> 




More information about the dev mailing list