diff options
author | Peter Wu <peter@lekensteyn.nl> | 2017-03-30 13:26:56 +0200 |
---|---|---|
committer | Peter Wu <peter@lekensteyn.nl> | 2017-03-31 09:17:42 +0000 |
commit | 641a48ec68778a5eaed82e436c09642e9ef1f911 (patch) | |
tree | 9b97002cd118761f444fa077632d34a084b081ba /caputils | |
parent | 08963526bc618cb827f7e1eddb547c3bea0d2017 (diff) | |
download | wireshark-641a48ec68778a5eaed82e436c09642e9ef1f911.tar.gz |
checkAPIs: improve trailing item suggestion for XXX_string
Tighten the check (do not just check for "0,NULL}};" but also
"{0,NULL}};" in case someone decides to use "{0x10,NULL}};" as trailing
item. Improve the suggestion, "{NULL,NULL}" is better for string_string
than "{0,NULL}".
For now treat 0 the same as NULL since files like packet-fix.h would
need editing otherwise. Accept octal notation for value_string since
packet-nfs.c uses this in nfs2_mode_names.
Change-Id: Ic507dbd8b07f3ae062b3f0310aa3398115d54273
Reviewed-on: https://code.wireshark.org/review/20796
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'caputils')
0 files changed, 0 insertions, 0 deletions