Wireshark-bugs: [Wireshark-bugs] [Bug 4887] Adding support for Gigamon headers (timestamp, sourc
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Thu, 17 Jun 2010 18:41:02 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4887

Stig Bjørlykke <stig@xxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |stig@xxxxxxxxxxxxx

--- Comment #3 from Stig Bjørlykke <stig@xxxxxxxxxxxxx> 2010-06-17 18:41:01 PDT ---
Hi,

1. Do you need to add another hard-wired column for the gigamon time?  I think
this should be fetched from the Gigamon Header and presented in a custom
column.  This way we do not need to handle this time in the frame dissector,
column utils and a lot of other places.

2. This patch contains a lot of whitespace changes, which makes it hard to
read.  This should be separated in its own patch if this is a whitespace fix.

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