Wireshark-commits: [Wireshark-commits] master bded74e: Do not use #if* in struct
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 21 May 2018 12:05:35 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bded74eb669a5d1789c2ea876e0044db17fe61e2
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

bded74e by Michał Łabędzki (michal.labedzki@xxxxxxxxxxxxx):

    Do not use #if* in struct
    
    Do not assume that somebody always includes "config.h" with "epan/prefs.h",
    because I do not include that one and I notice a lot of bugs, like:
    in one file sizeof(prefs) is 538 but in another is 544...
    and of course preferences do not work as expected - order of fields
    in the struct was changed. Try to avoid #if in struct.
    
    Remove dependency of config.h for prefs by removing #ifs, because all prefs
    should be registered also if not used.
    
    Change-Id: I02f71d2e64acf3e0f90c802b1610cc4620727051
    Reviewed-on: https://code.wireshark.org/review/27654
    Petri-Dish: Michal Labedzki <michal.labedzki@xxxxxxxxxxxxx>
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  9508205   json: fix typo.
    adds  bded74e   Do not use #if* in struct


Summary of changes:
 epan/prefs.c | 7 -------
 epan/prefs.h | 4 ----
 2 files changed, 11 deletions(-)