Wireshark-commits: [Wireshark-commits] master e60b8d1: dot11decrypt: Shorten the debug macros
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e60b8d1eb11e1bef6159534b4f3e45a0f6a3bf3b
Submitter: "Roland Knall <rknall@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark
Commits:
e60b8d1 by Mikael Kanstrup (mikael.kanstrup@xxxxxxxx):
dot11decrypt: Shorten the debug macros
The debug log macros are really long and require function name as
one parameter. This makes debug log lines either too wide or span
several lines of source code. Shorten the macro defines and make
use of G_STRFUNC to avoid manual function name entries in code.
NOTE: A bonus of removing all the manual function name entries is
that browsing/searching the code for function names is much easier.
Change-Id: Ia643f56df76e4a1b01ee6e6818cd61ec01047d33
Reviewed-on: https://code.wireshark.org/review/34927
Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
Actions performed:
from eeb179e snort: fix non-static 'g_file_separator' variable
add e60b8d1 dot11decrypt: Shorten the debug macros
Summary of changes:
epan/crypt/dot11decrypt.c | 248 ++++++++++++++++++++--------------------
epan/crypt/dot11decrypt_debug.h | 66 +++--------
2 files changed, 143 insertions(+), 171 deletions(-)