Wireshark-commits: [Wireshark-commits] master e60b8d1: dot11decrypt: Shorten the debug macros
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 03 Nov 2019 17:20:52 +0000
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(-)