tcpdump mailing list archives
Re: [Patch] print-zeromq.c
From: "Gisle Vanem" <gvanem () yahoo no>
Date: Tue, 4 Mar 2014 15:48:12 +0100
"François-Xavier Le Bail" <fx.lebail () yahoo com> wrote:
Is it not better to add const in the propotype ?
Comparing to other print-function with a 'len' as last parameter, I think not. Looking through interface.h for *_print() functions, only these have a 'const u_int' as last arg: openflow_print, ahcp_print. IMHO, we should be consistent; a 'const u_int len' or simply a 'u_int len'. Some even have a 'register' modifier which seems odd. --gv _______________________________________________ tcpdump-workers mailing list tcpdump-workers () lists tcpdump org https://lists.sandelman.ca/mailman/listinfo/tcpdump-workers
Current thread:
- [Patch] print-zeromq.c Gisle Vanem (Mar 04)
- Re: [Patch] print-zeromq.c François-Xavier Le Bail (Mar 04)
- Re: [Patch] print-zeromq.c Gisle Vanem (Mar 04)
- Re: [Patch] print-zeromq.c François-Xavier Le Bail (Mar 05)
- Re: [Patch] print-zeromq.c Guy Harris (Mar 04)
- Re: [Patch] print-zeromq.c François-Xavier Le Bail (Mar 05)
- Re: [Patch] print-zeromq.c Gisle Vanem (Mar 04)
- Re: [Patch] print-zeromq.c François-Xavier Le Bail (Mar 04)