[ovs-dev] [flow monitor v2 02/12] ofproto: Mark some function parameters const.

[Justin Pettit]

Looks good.

--Justin


On Jul 6, 2012, at 2:49 PM, Ben Pfaff wrote:

> This allows FOR_EACH_MATCHING_TABLE to be used with const TABLE and
> OFPROTO arguments.
> 
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> ofproto/ofproto.c |    8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
> index a5e1d39..a710b3d 100644
> --- a/ofproto/ofproto.c
> +++ b/ofproto/ofproto.c
> @@ -2296,7 +2296,7 @@ check_table_id(const struct ofproto *ofproto, uint8_t table_id)
> }
> 
> static struct oftable *
> -next_visible_table(struct ofproto *ofproto, uint8_t table_id)
> +next_visible_table(const struct ofproto *ofproto, uint8_t table_id)
> {
>     struct oftable *table;
> 
> @@ -2312,7 +2312,7 @@ next_visible_table(struct ofproto *ofproto, uint8_t table_id)
> }
> 
> static struct oftable *
> -first_matching_table(struct ofproto *ofproto, uint8_t table_id)
> +first_matching_table(const struct ofproto *ofproto, uint8_t table_id)
> {
>     if (table_id == 0xff) {
>         return next_visible_table(ofproto, 0);
> @@ -2324,8 +2324,8 @@ first_matching_table(struct ofproto *ofproto, uint8_t table_id)
> }
> 
> static struct oftable *
> -next_matching_table(struct ofproto *ofproto,
> -                    struct oftable *table, uint8_t table_id)
> +next_matching_table(const struct ofproto *ofproto,
> +                    const struct oftable *table, uint8_t table_id)
> {
>     return (table_id == 0xff
>             ? next_visible_table(ofproto, (table - ofproto->tables) + 1)
> -- 
> 1.7.2.5
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev