Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 2816] New: Simplifying column addition.

Date: Mon, 18 Aug 2008 20:42:02 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2816

           Summary: Simplifying column addition.
           Product: Wireshark
           Version: 1.0.2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Low
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: ravikondamuru@xxxxxxxxx



Ravi Kondamuru <ravikondamuru@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2168|                            |review_for_checkin?
               Flag|                            |


Created an attachment (id=2168)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=2168)
Patch contains the code to declare column info once 

Build Information:
wireshark 1.0.2 (SVN Rev 25786)

Copyright 1998-2008 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled with GTK+ 2.12.8, with GLib 2.14.6, with WinPcap (version unknown),
with libz 1.2.3, without POSIX capabilities, with libpcre 7.0, with SMI 0.4.8,
with ADNS, with Lua 5.1, with GnuTLS 2.3.8, with Gcrypt 1.4.1, with MIT
Kerberos, with PortAudio V19-devel, with AirPcap.

Running on Windows XP Service Pack 2, build 2600, with WinPcap version 4.0.2
(packet.dll version 4.0.0.1040), based on libpcap version 0.9.5, without
AirPcap.

Built using Microsoft Visual C++ 6.0

--
I was trying to add new columns and thought I could ease the declaration of
clumn format enums, slist and dlist. The current approach requires that 3
arrays in column.c/column_info.h be carefully edited to get the right behavior.
I have a patch to declare everything in one place and use it where needed. 

I was wondering if you will be interested in reviewing the change to accept for
use in the wireshark code base.


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