Wireshark-commits: [Wireshark-commits] master 80e6f62: Get rid of calls to ctype.h functions.
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: Mon, 20 Oct 2014 06:15:21 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=80e6f6251e96dd7b8a9ab0fdbf63b03a73ef6692
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

80e6f62 by Guy Harris (guy@xxxxxxxxxxxx):

    Get rid of calls to ctype.h functions.
    
    They don't handle values outside the range -1 to 127, and their behavior
    is locale-dependent.  Use g_ascii_isXXX() and g_ascii_toXXX() instead of
    isXXX() and toXXX().
    
    If you're checking for printable ASCII, don't use isascii() and don't
    use iscntrl(), use g_ascii_isprint().  If you're checking for graphical
    ASCII, i.e. printable ASCII except for a space, use g_ascii_isgraph().
    
    Use ws_xton() to convert a hex digit character to the corresponding
    numeric value.
    
    Change-Id: Id3039bc586fbf66d8736c2df248c790c0d7a2330
    Reviewed-on: https://code.wireshark.org/review/4851
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  0b9eb9f   Get rid of unnecessary includes of ctype.h.
    adds  80e6f62   Get rid of calls to ctype.h functions.


Summary of changes:
 epan/dissectors/packet-aprs.c             |    9 +++----
 epan/dissectors/packet-beep.c             |    3 +--
 epan/dissectors/packet-bthfp.c            |    6 ++---
 epan/dissectors/packet-bthsp.c            |    6 ++---
 epan/dissectors/packet-catapult-dct2000.c |    7 +++--
 epan/dissectors/packet-cups.c             |   16 +++---------
 epan/dissectors/packet-diameter.c         |    3 +--
 epan/dissectors/packet-ftp.c              |   13 +++++-----
 epan/dissectors/packet-giop.c             |   40 +++--------------------------
 epan/dissectors/packet-http.c             |   17 ++++--------
 epan/dissectors/packet-icap.c             |   18 ++++---------
 epan/dissectors/packet-imap.c             |    7 +++--
 epan/dissectors/packet-imf.c              |   16 +++++-------
 epan/dissectors/packet-irc.c              |   25 +++++++++---------
 epan/dissectors/packet-megaco.c           |   15 +++++------
 epan/dissectors/packet-mgcp.c             |    5 ++--
 epan/dissectors/packet-mrcpv2.c           |    3 +--
 epan/dissectors/packet-msrp.c             |    7 +++--
 epan/dissectors/packet-multipart.c        |    5 ++--
 epan/dissectors/packet-ntlmssp.c          |    5 ++--
 epan/dissectors/packet-pvfs2.c            |    3 +--
 epan/dissectors/packet-rtpproxy.c         |    3 +--
 epan/dissectors/packet-rtsp.c             |   37 +++++++++++---------------
 epan/dissectors/packet-sip.c              |   22 ++++++++--------
 epan/dissectors/packet-smb-pipe.c         |    5 ++--
 epan/dissectors/packet-smtp.c             |    5 ++--
 epan/dissectors/packet-spdy.c             |    8 +++---
 epan/dissectors/packet-syslog.c           |    3 +--
 epan/dissectors/packet-tpkt.c             |    6 ++---
 epan/dissectors/packet-xcsl.c             |    5 ++--
 30 files changed, 114 insertions(+), 209 deletions(-)