diff options
| author | Guy Harris <guy@alum.mit.edu> | 2000-11-19 08:54:37 +0000 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2000-11-19 08:54:37 +0000 |
| commit | 252d55d80f92fa8267758fbf4faab520d2f79273 (patch) | |
| tree | b6449cf609799ebb75b91e1cd6b600b06ee3d11a /packet-wtp.c | |
| parent | e88bd04f9aa217173cc86d785e9206e272a832ce (diff) | |
| download | wireshark-252d55d80f92fa8267758fbf4faab520d2f79273.tar.gz wireshark-252d55d80f92fa8267758fbf4faab520d2f79273.tar.bz2 wireshark-252d55d80f92fa8267758fbf4faab520d2f79273.zip | |
For each column, have both a buffer into which strings for that column
can be put, and a pointer to the string for the column, which might or
might not point to that buffer.
Add a routine "col_set_str()", which sets the string for the column to
the string passed to it as an argument; it should only be handed a
static string (a string constant would be ideal). It doesn't do any
copying, so it's faster than "col_add_str()".
Make the routines that append to columns check whether the pointer to
the string for the column points to the buffer for the column and, if
not, copy the string for the column to the buffer for the column so that
you can append to it (so you can use "col_set_str()" and then use
"col_append_str()" or "col_append_fstr()").
Convert a bunch of "col_add_str()" calls that take a string constant as
an argument to "col_set_str()" calls.
Convert some "col_add_fstr()" calls that take a string constant as the
only argument - i.e., the format string doesn't have any "%" slots into
which to put strings for subsequent arguments to "col_set_str()" calls
(those calls are just like "col_add_str()" calls).
Replace an END_OF_FRAME reference in a tvbuffified dissector with a
"tvb_length(tvb)" call.
svn path=/trunk/; revision=2670
Diffstat (limited to 'packet-wtp.c')
| -rw-r--r-- | packet-wtp.c | 13 |
1 files changed, 2 insertions, 11 deletions
diff --git a/packet-wtp.c b/packet-wtp.c index eab342b6be..99566bf7aa 100644 --- a/packet-wtp.c +++ b/packet-wtp.c @@ -3,7 +3,7 @@ * * Routines to dissect WTP component of WAP traffic. * - * $Id: packet-wtp.c,v 1.3 2000/11/05 09:30:11 guy Exp $ + * $Id: packet-wtp.c,v 1.4 2000/11/19 08:54:10 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -264,9 +264,7 @@ dissect_wtp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) /* This field shows up as the "Info" column in the display; you should make it, if possible, summarize what's in the packet, so that a user looking - at the list of packets can tell what type of packet it is. - "col_add_fstr()" can be used instead of "col_add_str()"; it takes - "printf()"-like arguments. */ + at the list of packets can tell what type of packet it is. */ if (check_col(fdata, COL_INFO)) { #ifdef DEBUG fprintf( stderr, "dissect_wtp: (6) About to set info_col header to %s\n", szInfo ); @@ -276,13 +274,6 @@ dissect_wtp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) /* In the interest of speed, if "tree" is NULL, don't do any work not necessary to generate protocol tree items. */ if (tree) { -/* NOTE: The offset and length values in the previous call to - "proto_tree_add_item()" define what data bytes to highlight in the hex - display window when the line in the protocol tree display - corresponding to that item is selected. - - END_OF_FRAME is a handy way to highlight all data from the offset to - the end of the packet. */ #ifdef DEBUG fprintf( stderr, "dissect_wtp: cbHeader = %d\n", cbHeader ); #endif |