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

Wireshark-dev: Re: [Wireshark-dev] Remove old packet list code in trunk?

From: Jaap Keuter <jaap.keuter@xxxxxxxxx>
Date: Wed, 18 Aug 2010 13:43:12 +0200
On Wed, 18 Aug 2010 11:14:43 +0200, Anders Broman
<anders.broman@xxxxxxxxxxxx> wrote:
>>-----Original Message-----
>>From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Jaap Keuter
>>Sent: den 18 augusti 2010 11:02
>>To: Developer support list for Wireshark
>>Subject: Re: [Wireshark-dev] Remove old packet list code in trunk?
>>
>>Hi,
>>
>>Mind you that it would make back porting to trunk-1.4 of packet list related code more difficult since the
>>source code will diverge.
>>That means more manual back porting, which is more laborious and error prone (sorry Gerald).
>>
>>I would suggest holding off on stripping trunk of old packet list code, but not to maintain it. After we put >1.4.1 'in the wild' and survives real user scrutiny it time to start ripping it out IMHO.
>>
>>Thanks,
>>Jaap
> 
> I agree with you, just to be clear not having NEW_PACKET_LIST defined
> is not expected to
> Work in trunk any more. Perhapps adding comments about that and/or
> change it to #if 0 if edditing a file for
> Other reasons?
> Regards
> Anders 
> 

Hi,

The only thing to be done in trunk is to remove the configuration
option or add a 'no longer functional' remark to the item
Changing to #if 0 would also invalidate patches for back porting.

Thanks,
Jaap

>>
>>On Wed, 18 Aug 2010 07:40:08 +0200, Anders Broman <a.broman@xxxxxxxxx>
>>wrote:
>>> Hi,
>>> In bug 5115 the question is raised if we should remove the "old packet
>>> list" code as it becomes cumbersome to write code for both branches in
>>> some cases. With the upcoming release of 1.4 I suppose we might as
>>> well remove the code in trunk.
>>> Regards
>>> Anders
>>>