Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 44435: /trunk/epan/ /trunk/epan/diss

From: Pascal Quantin <pascal.quantin@xxxxxxxxx>
Date: Sat, 11 Aug 2012 10:05:51 +0200
Le 11 août 2012 à 08:26, Joerg Mayer <jmayer@xxxxxxxxx> a écrit :

> On Fri, Aug 10, 2012 at 08:33:02PM +0000, gerald@xxxxxxxxxxxxx wrote:
>> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=44435
>> 
>> User: gerald
>> Date: 2012/08/10 01:33 PM
>> 
>> Log:
>> Make the corresponding packet_info available to each tree item. This
>> lets us pass a NULL pinfo to expert_add_info_format() and
>> expert_add_undecoded_item(), which makes it possible to use those
>> routines deep in the bowels of many dissectors. As a proof of concept
>> remove the recent pinfo additions to packet-afp.c. This should also make
>> it easier to fix bug 3884.
> 
> Does this parameter have any use left? If not we could and should remove it
> throughout the source including the function header.

Hi,
according to my understanding pinfo can be NULL if pi is not NULL. So most of the changes done in r44441 will lead to an expert info not being displayed at all.

Regards,
Pascal.