Alexis La Goutte
changed
bug 9219
| What |
Removed |
Added |
| Attachment #11699 Flags |
review_for_checkin?
|
review_for_checkin-
|
Comment # 3
on bug 9219
from Alexis La Goutte
Comment on attachment 11699 [details]
patch
(In reply to comment #2)
> (Knowing nothing about 'zebra' and not having looked at the code or the
> patch)
>
> Will the patched dissector still be able to dissect previous 'zebra'
> captures ?
>
> That is: Is the new protocol version backwards compatible with the old or
> what ?
+ 1 ?
There is also some error (found by check* tools) :
Error: the blurb for hf_zebra_routeridmask ("zebra.routerIDMask") matches the
field name in epan/dissectors/packet-zebra.c
Error: the blurb for hf_zebra_mac ("zebra.macaddress") matches the field name
in epan/dissectors/packet-zebra.c
Error: Found C++ style comments in epan/dissectors/packet-zebra.c
And also some trailing whitespace !
You are receiving this mail because:
- You are watching all bug changes.