Wireshark-commits: [Wireshark-commits] master c65e5851: Make some items that don't need to be size_
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: Thu, 15 Dec 2016 11:53:15 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c65e5851b2eef0ef99e4cb9388eb780cd6a87aa7
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c65e5851 by Guy Harris (guy@xxxxxxxxxxxx):

    Make some items that don't need to be size_t guint.
    
    Those sizes are limited by the packet sizes we support, and we only
    support a maximum packet size of 2^32.
    
    This squelches some compiler warnings.
    
    Remove some casts that this renders unnecessary.
    
    Change-Id: Id9a7bcf8c2ce30bbed7be6c0e28deb9cf38002e0
    Reviewed-on: https://code.wireshark.org/review/19279
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  e6a9877   Improved  parsing of sniffed homeplug-av data
    adds  c65e5851  Make some items that don't need to be size_t guint.


Summary of changes:
 epan/dissectors/packet-rtp.h |    4 ++--
 ui/gtk/rtp_analysis.c        |    2 +-
 ui/qt/rtp_audio_stream.cpp   |    2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)