On Sun, Dec 18, 2016 at 12:18 AM, Ben Pfaff <b...@ovn.org> wrote:
> The 'tc' member of struct ovsdb_idl_condition was written but never read,
> so remove it.
>
> Signed-off-by: Ben Pfaff <b...@ovn.org>
>
Acked-by: Mickey Spiegel<mickeys....@gmail.com>
---
> lib/ovsdb-idl-provider.h | 1 -
> lib/ovsdb-idl.c | 9 +++------
> 2 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/lib/ovsdb-idl-provider.h b/lib/ovsdb-idl-provider.h
> index 3104f2c..1f5a49e 100644
> --- a/lib/ovsdb-idl-provider.h
> +++ b/lib/ovsdb-idl-provider.h
> @@ -103,7 +103,6 @@ struct ovsdb_idl_table_class {
> };
>
> struct ovsdb_idl_condition {
> - const struct ovsdb_idl_table_class *tc;
> struct ovs_list clauses;
> };
>
> diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
> index 8ce228d..3974eb6 100644
> --- a/lib/ovsdb-idl.c
> +++ b/lib/ovsdb-idl.c
> @@ -219,8 +219,7 @@ ovsdb_idl_table_from_class(const struct ovsdb_idl *,
> const struct ovsdb_idl_table_class *);
> static bool ovsdb_idl_track_is_set(struct ovsdb_idl_table *table);
> static void ovsdb_idl_send_cond_change(struct ovsdb_idl *idl);
> -static void ovsdb_idl_condition_init(struct ovsdb_idl_condition *cnd,
> - const struct ovsdb_idl_table_class
> *tc);
> +static void ovsdb_idl_condition_init(struct ovsdb_idl_condition *cnd);
>
> /* Creates and returns a connection to database 'remote', which should be
> in a
> * form acceptable to jsonrpc_session_open(). The connection will
> maintain an
> @@ -279,7 +278,7 @@ ovsdb_idl_create(const char *remote, const struct
> ovsdb_idl_class *class,
> = table->change_seqno[OVSDB_IDL_CHANGE_MODIFY]
> = table->change_seqno[OVSDB_IDL_CHANGE_DELETE] = 0;
> table->idl = idl;
> - ovsdb_idl_condition_init(&table->condition, tc);
> + ovsdb_idl_condition_init(&table->condition);
> table->cond_changed = false;
> }
>
> @@ -892,10 +891,8 @@ ovsdb_idl_condition_reset(struct ovsdb_idl *idl,
> }
>
> static void
> -ovsdb_idl_condition_init(struct ovsdb_idl_condition *cnd,
> - const struct ovsdb_idl_table_class *tc)
> +ovsdb_idl_condition_init(struct ovsdb_idl_condition *cnd)
> {
> - cnd->tc = tc;
> ovs_list_init(&cnd->clauses);
> }
>
> --
> 2.10.2
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
