Wireshark-commits: [Wireshark-commits] master 5bfde7c: Don't use ctype.h routines.
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: Fri, 17 Oct 2014 20:37:45 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5bfde7c6386b56db80fec85e2ca7dc1d271b461c
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

5bfde7c by Guy Harris (guy@xxxxxxxxxxxx):

    Don't use ctype.h routines.
    
    That avoids locale dependency and handles possibly-signed chars (which
    we weren't doing before).
    
    Change-Id: I70f3d993c9a8fbf870901f12b430d733968c3fa8
    Reviewed-on: https://code.wireshark.org/review/4781
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  35a9b98   Use proto_tree_add_bitmask for ExpandedNodeId EncodingMask
    adds  5bfde7c   Don't use ctype.h routines.


Summary of changes:
 wiretap/catapult_dct2000.c |   27 +++++++++++++--------------
 wiretap/cosine.c           |    3 +--
 wiretap/daintree-sna.c     |   13 ++++++-------
 wiretap/dbs-etherwatch.c   |   19 +++++++++----------
 wiretap/dct3trace.c        |    3 +--
 wiretap/iseries.c          |    5 ++---
 wiretap/netscreen.c        |    5 ++---
 wiretap/vms.c              |   11 +++++------
 8 files changed, 39 insertions(+), 47 deletions(-)