Wireshark · Wireshark-dev: Re: [Wireshark-dev] [PATCH] New menu items to copy packet data

[Stephen Fisher <stephentfisher@xxxxxxxxx>]

On Thu, Jan 11, 2007 at 01:59:56PM -0000, Douglas Pratley wrote:

> This is a patch adding items to the context menus for the packet list 
> and packet details panes for copying packet data to the clipboard.

Looking good so far.  I fixed a few warnings about unused variables.  
BTW, if you have a function call that takes a parameter that it doesn't 
use you can put _U_ after it and Wireshark's compilation process will 
expand that to code that tells (gcc at least) not to warn about that.

> I'm not convinced that Base64 is useful; I wanted to implement a 
> non-trivial encoding to test a particular path through the code.

Instead of encoding the contents into base64, decoding a stream of 
base64 data and placing the decoded data in the clipboard would be very 
useful.

> I have not updated any documentation because I cannot find a 
> description of what context menu items do anywhere (main menu items 
> are listed in the man page). If I should have done, please let me 
> know.

The documentation here needs to be updated with new screenshots and 
text: 
http://www.wireshark.org/docs/wsug_html_chunked/ChWorkDisplayPopUpSection.html 
You can find this chapter in the source as 
docbook/wsug_src/WSUG_chapter_work.xml.


Steve