Wireshark-bugs: [Wireshark-bugs] [Bug 4014] ICMP: Add ID and seq # to Info column

Date: Tue, 14 Sep 2010 14:40:39 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4014

--- Comment #18 from Stephen Fisher <steve@xxxxxxxxxxxxxxxxxx> 2010-09-14 15:40:36 MDT ---
(In reply to comment #17)
> (In reply to comment #16)
> > (In reply to comment #10)
> > > I think the info column gets a bit messy with this info.
> > 
> > How about we remove some of the characters to change it from:
> > 
> >   (id=0x412b, seq(be/le)=1/256, ttl=64)
> > 
> > To
> > 
> >   id=0x412b, seq=1/256, ttl=64
> 
> +1 for me :-)

Committed in SVN revision 34119.  I also added a little endian representation
of the identifier field following the pattern of the sequence number.

> I would vote yes to that question. You can quickly see in the info column which
> packets belong to one-another when mutiple pings are in the trace. To me the
> representation is unimportant, as long as it's consistent. It does not have a
> numerical significance (to me).

Makes sense.  I left the info column as a hex display in the (Wireshark) host's
byte order since that's probably easier to read and compare than decimal.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.