Hi,

... and if indeed it needs to be removed the documentation (README.developer) 
must be adapted as well on this API change.

Thanx,
Jaap

Guy Harris wrote:
> On Dec 18, 2008, at 1:03 PM, Martin Mathieson wrote:
> 
>> Sorry about that, I didn't grep to see if it was being called.   
>> Because it compiled OK I wrongly assumed it wasn't.
> 
> BTW, as it's an add_bytes routine, presumably it should take a const  
> guint8 * pointing to the sequence of bytes to be added, the way  
> proto_tree_add_bytes() and proto_tree_add_bytes_format_value() do.   
> Why was the start_ptr argument removed?

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to