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 <jrajaha...@nicira.com>
Pushed to master,
Jarno
On Sep 10, 2014, at 1:28 AM, Daniele Venturino <daniele.ventur...@m3s.it> wrote:
> Signed-off by: Daniele Venturino <daniele.ventur...@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
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
