Wireshark-commits: [Wireshark-commits] master 39ec5f8: CMake: allow nested structures to be initial
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 09 Dec 2018 07:13:22 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=39ec5f8ec20a5ee0c916629e4556ab16270f0208
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

39ec5f8 by Peter Wu (peter@xxxxxxxxxxxxx):

    CMake: allow nested structures to be initialized via { 0 }
    
    This is already permitted since C89 and popular compilers seem to
    support it (including Clang 3.0 - 7.0, GCC 4.1.2 - 8.2 and MSVC 2015).
    GCC used to be buggy: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53119
    
    C89 6.5.7. Initialization: "only enough initializers from the list are
    taken to account for the members of the subaggregate".
    
    Change-Id: Ic59b9fe71e2d3ce60b4b7d1074f8d84af01a817a
    Reviewed-on: https://code.wireshark.org/review/30968
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  4160f10   proto: make ws_type_to_elastic static.
     add  39ec5f8   CMake: allow nested structures to be initialized via { 0 }


Summary of changes:
 CMakeLists.txt | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)