On 2015-11-04 at 17:20:35 +0100, Tobias Klauser <[email protected]> wrote:
> On 2015-11-03 at 19:06:49 +0100, Vadim Kochan <[email protected]> wrote:
> > Renamed flow_entry_direction to flow_direction to be
> > little shorter.
> > 
> > Changed enum value names to be in upper case.
> > 
> > Signed-off-by: Vadim Kochan <[email protected]>
> > ---
> >  flowtop.c | 36 ++++++++++++++++++------------------
> >  1 file changed, 18 insertions(+), 18 deletions(-)
> > 
> > diff --git a/flowtop.c b/flowtop.c
> > index 54a749a..6496e44 100644
> > --- a/flowtop.c
> > +++ b/flowtop.c
> > @@ -99,6 +99,14 @@ struct flow_list {
> >  #define INCLUDE_ICMP       (1 << 5)
> >  #define INCLUDE_SCTP       (1 << 6)
> >  
> > +enum flow_direction {
> > +   FLOW_DIR_SRC,
> > +   FLOW_DIR_DST,
> > +};
> > +
> > +#define SELFLD(dir,src_member,dst_member)  \
> > +   (((dir) == FLOW_DIR_SRC) ? n->src_member : n->dst_member)
> > +
> 
> No need to move these up here.

Actually, the enum makes sense at the top, so I left it there. The macro
is only use quite locally, so I left it there.

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to