[ovs-dev] [STP 7/9] ofproto: Mark 'ofproto' arg in is_mirror_output_bundle() as const.

Justin Pettit jpettit at nicira.com
Tue Oct 18 05:31:47 UTC 2011


No changes are made to 'ofproto', so it's safe to mark the argument as
const.  This will be useful in a later commit.
---
 ofproto/ofproto-dpif.c     |    2 +-
 ofproto/ofproto-provider.h |    2 +-
 ofproto/ofproto.c          |    2 +-
 ofproto/ofproto.h          |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 3de1807..d6bd2a5 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -1543,7 +1543,7 @@ set_flood_vlans(struct ofproto *ofproto_, unsigned long *flood_vlans)
 }
 
 static bool
-is_mirror_output_bundle(struct ofproto *ofproto_, void *aux)
+is_mirror_output_bundle(const struct ofproto *ofproto_, void *aux)
 {
     struct ofproto_dpif *ofproto = ofproto_dpif_cast(ofproto_);
     struct ofbundle *bundle = bundle_lookup(ofproto, aux);
diff --git a/ofproto/ofproto-provider.h b/ofproto/ofproto-provider.h
index f596abc..c571f52 100644
--- a/ofproto/ofproto-provider.h
+++ b/ofproto/ofproto-provider.h
@@ -948,7 +948,7 @@ struct ofproto_class {
 
     /* Returns true if 'aux' is a registered bundle that is currently in use as
      * the output for a mirror. */
-    bool (*is_mirror_output_bundle)(struct ofproto *ofproto, void *aux);
+    bool (*is_mirror_output_bundle)(const struct ofproto *ofproto, void *aux);
 
     /* When the configuration option of forward_bpdu changes, this function
      * will be invoked. */
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 92800cb..bbda478 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -658,7 +658,7 @@ ofproto_set_flood_vlans(struct ofproto *ofproto, unsigned long *flood_vlans)
 /* Returns true if 'aux' is a registered bundle that is currently in use as the
  * output for a mirror. */
 bool
-ofproto_is_mirror_output_bundle(struct ofproto *ofproto, void *aux)
+ofproto_is_mirror_output_bundle(const struct ofproto *ofproto, void *aux)
 {
     return (ofproto->ofproto_class->is_mirror_output_bundle
             ? ofproto->ofproto_class->is_mirror_output_bundle(ofproto, aux)
diff --git a/ofproto/ofproto.h b/ofproto/ofproto.h
index 1f39a25..9a8f755 100644
--- a/ofproto/ofproto.h
+++ b/ofproto/ofproto.h
@@ -249,7 +249,7 @@ int ofproto_mirror_register(struct ofproto *, void *aux,
 int ofproto_mirror_unregister(struct ofproto *, void *aux);
 
 int ofproto_set_flood_vlans(struct ofproto *, unsigned long *flood_vlans);
-bool ofproto_is_mirror_output_bundle(struct ofproto *, void *aux);
+bool ofproto_is_mirror_output_bundle(const struct ofproto *, void *aux);
 
 /* Configuration querying. */
 bool ofproto_has_snoops(const struct ofproto *);
-- 
1.7.1




More information about the dev mailing list