Wireshark-bugs: [Wireshark-bugs] [Bug 1957] New Dissector: EPCglobal Low-Level Reader Protocol
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1957
------- Comment #11 from matt.poduska@xxxxxxxxxxxx 2007-12-17 23:15 GMT -------
Thanks for the info:
1) In the license part, two lines seem to be duplicated.
Which file? I removed the COPYING file from the llrp directory.
2) Updates to the Unix build process are missing (configure.in, Makefile.am).
Fixed
3) Why the ifdef cplusplus in the header files?
The LLRP message parser is generic code that can be used in other projects (see
comment 5 in this bug entry).
4) Some files seem to be duplicated in the patch.
I saw that. I'm using TortiseSVN's "create patch" function, and I'm not sure
how to make it stop adding all of the files twice. Any suggestions?
5) The package name in moduleinfo.nmake seems to be wrong.
Fixed
--
Configure bugmail: http://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
