diff options
author | Huanle Han <hanxueluo@gmail.com> | 2016-01-15 23:24:59 +0800 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-02-05 15:36:05 -0800 |
commit | 6c38bdc0c5bcc8b080fc689e436b35830d69eb75 (patch) | |
tree | 719ee3ef0e0f79d537a7cacf627c4a6290e632b3 /ofproto | |
parent | e0c73ed1310554d9326adbc54e1dd279e1ae8c1f (diff) | |
download | openvswitch-6c38bdc0c5bcc8b080fc689e436b35830d69eb75.tar.gz |
ofproto-dpif-xlate: Fix clash between macro argument and struct member name.
Argument name "entries" is same as the member "entries" of xcache.
It will lead to compile error if we pass an argument whose name is
not "entries".
Signed-off-by: Huanle Han <hanxueluo@gmail.com>
Diffstat (limited to 'ofproto')
-rw-r--r-- | ofproto/ofproto-dpif-xlate.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index 1edc1b0e8..a6ea0679d 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -457,11 +457,11 @@ struct xc_entry { } u; }; -#define XC_ENTRY_FOR_EACH(entry, entries, xcache) \ - entries = xcache->entries; \ - for (entry = ofpbuf_try_pull(&entries, sizeof *entry); \ - entry; \ - entry = ofpbuf_try_pull(&entries, sizeof *entry)) +#define XC_ENTRY_FOR_EACH(ENTRY, ENTRIES, XCACHE) \ + ENTRIES = XCACHE->entries; \ + for (ENTRY = ofpbuf_try_pull(&ENTRIES, sizeof *ENTRY); \ + ENTRY; \ + ENTRY = ofpbuf_try_pull(&ENTRIES, sizeof *ENTRY)) struct xlate_cache { struct ofpbuf entries; |