Andy Chu wrote: > Hi dev, > > The attachment is a plugin for China Mobile Point to Point (cmpp) > protocol, it's the SMS protocol used in China. I have test it on Mac > OS X and Fedora Core 5.
Sorry for the delay in reviewing your code. A few comments: - Would you mind converting this to a regular dissector (in epan/dissectors/) instead of a plugin? I, personally, am not a big fan of plugins. - Here: > /* CMPP_TERMINATE and CMPP_TERMINATE_RESP don't have > msg body */ > case 0x00000002: > break; > case 0x80000002: > break; you've got #defines for these numbers, why not use them (it'll make it more readable)? - Here: _______________________________________________ Wireshark-dev mailing list Wireshark-dev@wireshark.org http://www.wireshark.org/mailman/listinfo/wireshark-dev
