Jaap Keuter wrote:
> 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.
Would moving to Git or Mercurial help for this?