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: