On Sun, Jan 13, 2008 at 11:56:44AM +0000, [EMAIL PROTECTED] wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=24074
> 
> User: jake
> Date: 2008/01/13 11:56 AM
> 
> Log:
>  From Michael Earnhart:
>  Patch to add cdp Power_request and Power_available fields
>  This added the support for the 0x0019 and 0x001a power_request and 
> power_available fields.
>  
>  Submitted patch slightly modified to present summary line and make more 
> robust.

I've two open points with this patch:
1) Why don't you use proto_tree_add_item?
2) Could you please print the skipped bytes in case 0<power_avail_len<4
   as padding?


   Ciao
      Joerg
-- 
Joerg Mayer                                           <[EMAIL PROTECTED]>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
_______________________________________________
Wireshark-dev mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-dev

Reply via email to