Wireshark-commits: [Wireshark-commits] master 4f4769a: Move structure definition outside another st
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 13 Jan 2016 09:41:15 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4f4769adf626ec2601e3488efbc2e2d57d55762c
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

4f4769a by Guy Harris (guy@xxxxxxxxxxxx):

    Move structure definition outside another structure definition.
    
    I guess the ability to define a structure inside another structure is a
    C-ism discarded by C++, so it causes warnings if you disallow stuff that
    can't be handled by a C++ compiler, as we do.
    
    Change-Id: I8cf52af0424708eb663ab6dbfecbf317fe3bccdb
    Reviewed-on: https://code.wireshark.org/review/13257
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  5892686   Get rid of debugging printouts.
    adds  4f4769a   Move structure definition outside another structure definition.


Summary of changes:
 wiretap/wtap.h |   10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)